This PR introduces several new features and improvements, including:
Fetching and displaying top links on the dashboard.
Adding functionality to display tags in a dropdown and the current filters section located within the "Filter by Tag" section of the dashboard.
Integrates additional fetch call to retrieve top links based on selected tag from the filter dropdown.
Error handling for missing tags and enhanced user experience
Adds navigation links to the navigation bar.
What are the relevant tickets (if any)
This PR is related to issue #16 #17, and will be closed upon successful merge of this PR request.
Screenshots (if applicable)
Testing
The dashboard data, layout and expected behavior will be tested in a separate testing branch. Local testing has been done to ensure there is adequate error handling and the application behaves as expected.
Checklist
[ ] The code follows the project's coding standards.
[ ] Unit tests have been added or updated to cover the changes.
[ ] Documentation has been updated to reflect the changes (if applicable).
[ ] The code compiles without errors.
[ ] The changes have been tested locally and pass all relevant tests.
[ ] All new and existing tests pass.
[ ] The pull request has been reviewed by at least one other contributor.
Reviewer Instructions
Please focus on reviewing the following:
The accuracy of the data fetching and display logic for tags and links on the dashboard.
Error handling implemented for missing tags, no results based on filters applied and fetching issues.
The overall user experience, including the display of filters and navigation links.
Deployment Notes
No specific deployment instructions are required for this PR.
[x] The accuracy of the data fetching and display logic for tags and links on the dashboard. I confirmed there are 5 links available in the backend. These 5 links are displayed on the dashboard page. I also confirmed that with the exception of the first link, no other links have tags. This matches with what is displayed on the frontend. The first link has numerous tags and the other four links have "no tags assigned for this link".
[x] Error handling implemented for missing tags, no results based on filters applied and fetching issues. *I confirmed the "No links found for the selected tag, please select another filter." is displayed to the user when there are no results based on the filters applied. Additionally, I modified the endpoint in the fetchTopLinks to confirm an manually invoke an error message. "No links found." is displayed to the user.**
[x] The overall user experience, including the display of filters and navigation links. I think the overall user experience is positive. I like the current filters in the underneath the Filter by Tag section.
Description of Changes
This PR introduces several new features and improvements, including:
What are the relevant tickets (if any)
This PR is related to issue #16 #17, and will be closed upon successful merge of this PR request.
Screenshots (if applicable)
Testing
The dashboard data, layout and expected behavior will be tested in a separate testing branch. Local testing has been done to ensure there is adequate error handling and the application behaves as expected.
Checklist
Reviewer Instructions
Please focus on reviewing the following:
Deployment Notes
No specific deployment instructions are required for this PR.
Additional Information
N/A