Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fixed all markdown issues #614

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

smritidoneria
Copy link
Contributor

Brief Title

Fixed all the message rendering issues.

Acceptance Criteria fulfillment

  • Task 1
  • Task 2
  • Task 3

Fixes #613

Video/Screenshots

Screen.Recording.2024-07-26.at.2.04.40.AM.mov

@devanshkansagra
Copy link
Contributor

@smritidoneria, Could you please check and share the result of strikethrough and other markdowns whether it is working or not when the quoted message is sent? When I saw your code, you have made no changes in attachment preview dir. So I think at that point there could be this small bug

@smritidoneria
Copy link
Contributor Author

hey @devanshkansagra , thanks for pointing out, I will look after it

@Spiral-Memory
Copy link
Collaborator

Hey @smritidoneria
Could you please resolve the conflicts?

@smritidoneria smritidoneria marked this pull request as ready for review October 22, 2024 14:12
@smritidoneria
Copy link
Contributor Author

hey @Spiral-Memory , changes have been made to resolve the conflicts, kindly look at it.

@Spiral-Memory
Copy link
Collaborator

Format check is failing @smritidoneria

@smritidoneria
Copy link
Contributor Author

hey @Spiral-Memory , Something is wrong with the built
SyntaxError: Unexpected identifier 'assert'
at compileSourceTextModule (node:internal/modules/esm/utils:337:16)
at ModuleLoader.moduleStrategy (node:internal/modules/esm/translators:166:18)
at callTranslator (node:internal/modules/esm/loader:416:14)
at ModuleLoader.moduleProvider (node:internal/modules/esm/loader:422:30)
at link (node:internal/modules/esm/module_job:88:21)

This error is coming,

I have ran prettier, installed babel for the Transpilation also , but it remains the same.

@Spiral-Memory
Copy link
Collaborator

Hey @smritidoneria,

It's working for all new PRs except yours. Could you please check if there's any issue with your setup?

If the problem still exists, I'll clone your code and investigate what might be causing it.

@smritidoneria
Copy link
Contributor Author

can u re-run the checks, I have resolved the errors @Spiral-Memory

@Spiral-Memory
Copy link
Collaborator

Format check passed, lint check failing

@smritidoneria
Copy link
Contributor Author

fixed.

@Spiral-Memory
Copy link
Collaborator

Can you please list all the places you fixed the markdown ? I need to verify

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

All markdown issues
3 participants