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

updated edit modal added backend for date edit #292

Merged
merged 3 commits into from
May 29, 2024

Conversation

Ansh-Garg22
Copy link
Contributor

I have added a feature that allows users to edit notes. When a user clicks on "Edit," a popup dialog box appears as an update modal, where they can modify the note's data. The backend now correctly updates the date when a note is edited, which was previously not functioning. Users can close the update modal by clicking the cross symbol if they wish to cancel.
This improves the UI of the page as well.

Similarly, the "Add Note" feature now appears as a popup dialog, providing a consistent user experience.

Hey @VaibhavArora314, please review the PR. Thank you!

FIXES #240

image
image

Copy link

Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. In the meantime, please ensure that your changes align with our CONTRIBUTING.md. If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊

@VaibhavArora314 VaibhavArora314 requested review from a team, Ultimateutkarsh11, goyalh4164 and VaibhavArora314 and removed request for a team May 26, 2024 09:41
@VaibhavArora314
Copy link
Collaborator

Resolve merge conflicts, rest looks good

@pallavisavant
Copy link
Collaborator

@Ansh-Garg22 you got some more conflicts to resolve. Thanks.

@Ansh-Garg22
Copy link
Contributor Author

Ansh-Garg22 commented May 29, 2024

Please merge as soon as possible because the conflicts arose due to interference from other code. Please merge the pull request.
I have resolved the conflicts
@pallavisavant @Ultimateutkarsh11

@pallavisavant pallavisavant merged commit 4feb265 into Note-Vault:main May 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
gssoc level2 GSSOC Level 2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feat: Improving Edit feature of Note
4 participants