-
-
Notifications
You must be signed in to change notification settings - Fork 122
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
Feat: Add the Newsletter feature to the website. #489
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you pls. resolve the failing lint checks. The suggestions below will be helpful for that : )
@DhairyaMajmudar not able to do that i am getting this After linting it. |
If you are using node version > 20 degrade it using with the help of node version manager , and then retry. |
@DhairyaMajmudar @benjagm done please check. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left 2 comments here.
Thanks a lot!!
Awesome job here !!! I pushed some changes to make sure it merges well into the dev branch!! Excellent! |
GitHub Issue: #243
Summary:
Do you think resolving this issue might require an Architectural Decision Record (ADR)? (significant or noteworthy)
No
No