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

[FE] Implement Glossary Page and Tag Type Pages #694

Merged
merged 3 commits into from
Dec 16, 2024

Conversation

asligook
Copy link
Contributor

📋 Proposed Changes

  • The glossary page displays 4 tag type cards for the tag types within the system, which are :

    • Programming Language
    • Software Library
    • Programming Paradigm
    • Computer Science Topic
  • Tag type cards include the name, short description and tag count of the tag type.

  • Tag type page includes the name, long description and tags in that tag type category.

  • Tag type information within the Tag Card component is turned into link and user is redirected to the related tag type page when clicked on.

Related Issue

Closes #652

🧪 Included Tests

  • Tag type cards are rendered correctly on the Glossary page with the correct information.
  • Tag type page includes the name, description and tags of the tag type correctly.
  • The user is redirected to the correct addresses within the application when clicked on the links on the newly implemented card components.

@asligook asligook added priority: high High priority component: frontend About frontend labels Dec 16, 2024
@asligook asligook requested a review from mmtftr December 16, 2024 15:02
@asligook asligook self-assigned this Dec 16, 2024
@asligook
Copy link
Contributor Author

I think I have committed my previous changes for this task at the same time with the another PR merge, so it appears like they were merged somehow before this PR.

Previous changes on this task can be found under this commit :
4cae533

Screenshot 2024-12-16 at 18 15 59

@asligook
Copy link
Contributor Author

Thanks for the PR review and approval @mmtftr, merging.

@asligook asligook merged commit 1e84295 into develop Dec 16, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: frontend About frontend priority: high High priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants