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

[Feature Implimented] Fixing Bug in Labels Card Functionality and Implimentation of Labels Page #226

Merged
merged 3 commits into from
Jun 25, 2023

Conversation

Rhishikesh12
Copy link
Contributor

Pull Request Template

Description:
This pull request resolves a bug in the labels card functionality, fixing the issue where clicking on it opens the wrong page (avatar_page) #217 . Additionally, it includes the implementation of the missing labels page.

Changes Made:

  1. Updated the function call in the labels card to point to the correct page.
  2. Implemented the labels functionality to display the appropriate labels for each item.

Related Issue:
#217

Screenshots/GIFs:

**Updated Function Call : **

image

Light Mode :

image

Dark Mode :

image

Checklist:

  • [✅] The code follows the project's coding style guidelines.

Please review and merge this PR at your earliest convenience.

Thank you!

@Rhishikesh12
Copy link
Contributor Author

Rhishikesh12 commented May 30, 2023

@ashdude1401 @Debajyoti14 No third-party libraries are used. Is this correct, or do I need to make any changes?

The assignment of this issue should be classified as Level 2, taking into consideration the implementation of a new feature.
I implemented the Labels page that was initially missing, so I expected my issue #217 to be assigned a Level 2 tag, considering the added functionality.

@Rhishikesh12 Rhishikesh12 changed the title Fixing Bug in Labels Card Functionality and Implimentation of Labels Page [Feature Implimented] Fixing Bug in Labels Card Functionality and Implimentation of Labels Page Jun 1, 2023
@Debajyoti14
Copy link
Member

@Rhishikesh12 Fix the conflicts and make the PR again

@Rhishikesh12
Copy link
Contributor Author

The assignment of this issue should be classified as Level 2, taking into consideration the implementation of a new feature. I implemented the Labels page that was initially missing, so I expected my issue #217 to be assigned a Level 2 tag, considering the added functionality.

Please have a look at tagged message and Solved the Merge Conflicts, Now you can merge.

@youhaveme9 youhaveme9 merged commit 826c668 into Clueless-Community:main Jun 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants