-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
bookmark feature #4
Comments
Hey @hkirat Your is is responding with 404 here, please checkout. |
Hello @hkirat, the screenshot image is not visible. |
Hey @hkirat can I please work on this??? |
Hey guys I am really new to this thing and i have a feeling that i can contribute somewhere and make my first contribution can u guys help me in making my first merge @devsargam & @AbdullahKhan08 |
@Vivek-1102 learn and then contribute. contributing without knowing stuff is not good for either you or the project. |
@shivamhwp I think i can do some of the stuffs but ur right i need to practise more than contributing i have still not manage to run the codebase locally can u help me to just to setup locally |
are you on 100xdevs discord. if not, join & i'll help you setup it locally. send me a dm @shivamhwp there |
Two things can be done here:
How can we proceed on this? |
@hkirat, is this issue still open? If so, I’d be happy to take it on. |
Issue is still open |
@Antiarin i'm working on it, if you want i will let you do it |
i have also, its quite simple but you continue, i will do it quickly and probably ur will be better or if u need help let me know |
Had started on working on this feature, but then the repo went AWOL for a while, and I got stuck and busy at work. Have completed the basic version of it, willl be completing the full feature if this looks good enough #237 |
Will require db migrations
The text was updated successfully, but these errors were encountered: