Skip to content

add and view notes with date time #10

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Harshit-crypto
Copy link

First of all this is an amazing repo and its very convenient to use!
Moving on to the pull request;
When i was solving problems filtered by a company , I wanted to quickly glance the approach I used to solve the problem but there was no way to do so apart from clicking solve and viewing the code again.
Hence, I added the notes feature to help me write short notes for each problem (which stores the same way as preferences and filters).

Hope you and other people find this useful!

Here's a screenshot to show how it looks
image

@nikhilm25
Copy link
Owner

i honestly didnt expect any pull requests.
im very happy to see people contributing.

i rightnow have a major push to make that is in progress, i will finish up with that and merge in your code

@nikhilm25
Copy link
Owner

i did test this,
it doesnt work

@nikhilm25 nikhilm25 closed this Jun 14, 2025
@Harshit-crypto
Copy link
Author

i did test this, it doesnt work

could you share details about the error?

@nikhilm25 nikhilm25 reopened this Jun 18, 2025
@nikhilm25
Copy link
Owner

im quite sorry there was an error on my end/

the functionality works fine.
A few improvements to be made would be an indicator to see whether a note has been made or how many notes have been added.
It is also misssing a bit of styling.

again sorry for my mistake.

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