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

Adds NodeJs + React Starter Template (NoteIt) #52

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

SoNiC-HeRE
Copy link

@SoNiC-HeRE SoNiC-HeRE commented Oct 27, 2023

My contribution for #35

A basic implementation of note taking app using React+NodeJs. More info is available here.

Please let me know if any changes are required

@naisofly
Copy link
Contributor

naisofly commented Nov 3, 2023

Hey @SoNiC-HeRE!

Can you kindly add a screenshot or GIF of the working app in your repo?
Also, I noticed if we try to delete any one note, all of them are deleted. Can you address this bug?

image

@SoNiC-HeRE
Copy link
Author

Hey @SoNiC-HeRE!

Can you kindly add a screenshot or GIF of the working app in your repo?
Also, I noticed if we try to delete any one note, all of them are deleted. Can you address this bug?

image

Sure, I'll make the necessary changes

@SoNiC-HeRE
Copy link
Author

Hie @naisofly, I've added the working video of the web-app to the readme of the repo.
However, I don't seem to face the bug you were facing, can you recheck please? and if it still persists can you please let me know how to reproduce it so I can get it fixed.

@naisofly
Copy link
Contributor

Hey @SoNiC-HeRE!

Apologies for the late response, but I tested it on another machine today with the latest release of SurrealDB and still see the same issue where all notes are deleted if you try to delete any single note — made a quick video here for your reference.

Hope this helps & I'm happy to merge this in once it's fixed 👍🏽

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