Feature/resource page #1
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
to add all the bookmarks in one place, created link and route /bookmarks in Header
Added BookmarkFlag icon an functionality
Update bookmarks table for better data integrity. Add bookmarks linking user_id to resource_id. Display user-specific bookmarks highlighted in the resource list.
Created a Bookmark Context: To manage shared bookmark state across components. Wrapped Application: Made the context available throughout the application. Updated Components: Modified BookmarksPage, HomePage, and ResourceList to utilise the context for managing bookmarks. Made a Commit: Saved and documented your changes.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is a: Bookmark resources implementation
[+] ✨ New feature - new behaviour has been implemented
🐛 Bug fix - existing behaviour has been made to behave
♻️ Refactor - the behaviour has not changed, just the implementation
✅ Test backfill - tests for existing behaviour were added but the behaviour itself hasn't changed
⚙️ Chore - maintenance task, behaviour and implementation haven't changed
Description
This pull request introduces a new bookmark management system that synchronises bookmarked resources across the Home page and the Bookmarks page, enhancing user experience by ensuring consistency.
Purpose -
This update allows users to efficiently manage their bookmarks, with changes reflected in real-time across different components of the application. Users can now toggle bookmarks on either the Home page or the Bookmarks page, and the changes will immediately be reflected in both locations.
How to check
To verify the functionality:
Log in to the application.
Navigate to the Home page and bookmark a resource by toggling the bookmark flag.
Then, go to the Bookmarks page to confirm that the newly bookmarked resource appears there.
Next, remove the bookmark from either page and check that it disappears from both the Home page and the Bookmarks page, confirming the real-time synchronization works as intended.
Links
CodeYourFuture#36
Author checklist
[+] I have written a title that reflects the relevant ticket
[+] I have written a description that says what the PR does and how to validate it
[+] I have linked to the project board ticket (and any related PRs/issues) in the Links section
[ ] I have added a link to this PR to the ticket
[+] I have made the PR to main from a branch named /, e.g. feature/edit-spaceships or bugfix/restore-oxygen
[+] I have manually tested that the app still works correctly
[+] I have requested reviewers here and in my team chat channel
[ ] I have spoken with my PM or TL about any parts of this task that may have become out-of-scope, or any additional improvements that I now realise may benefit my project
[ ] I have added tests, or new tests were not required
[ ] I have updated any documentation (e.g. diagrams, schemas), or documentation updates were not required
Screenshots: