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

Added Edit Snippet Functionality #11

Closed

Conversation

P-Bhaskar-Rao
Copy link

What does this PR do ?

Added Edit functionality so that users can alter their snippets

Fixes: #10

How should be this tested ?

  • view a snippet
  • At the top right corner , you can find a blue colored pen button which on hovering says "edit snippet". Click on it.
  • you can edit the snippet now and click update snippet button at the bottom to save your changes.
  • The page will redirect to the snippet and it contains updated data

Screenshots

edit-button

edit snippet page

Copy link

vercel bot commented Jan 4, 2025

@P-Bhaskar-Rao is attempting to deploy a commit to the suraniharsh's projects Team on Vercel.

A member of the Team first needs to authorize it.

@suraniharsh
Copy link
Owner

@P-Bhaskar-Rao

I'll create a separate branch for the edit snippet feature and submit a new pull request to that branch. Apologies for the delay, and thank you for your patience. Please let me know if any further changes are required.

@suraniharsh
Copy link
Owner

@P-Bhaskar-Rao

I’ve created a branch named feature/edit-snippets for this enhancement. Work is underway, and I’ll ensure the implementation aligns with the requirements. If you have additional input or suggestions, feel free to share them!

@suraniharsh suraniharsh closed this Jan 4, 2025
Copy link

vercel bot commented Jan 5, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
code-candy ❌ Failed (Inspect) Jan 5, 2025 5:43am

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.

2 participants