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

View For hacktoberfest labeled repo #1795

Merged
merged 4 commits into from
Oct 13, 2023
Merged

Conversation

avcwisesa
Copy link
Contributor

@avcwisesa avcwisesa commented Oct 9, 2023

Description

Added view for repos labeled Hacktoberfest. View is accessible from a dropdown in the navbar

Related Issue

#1680

Motivation and Context

Trying to tackle the linked issue (creating view for repos with some specific label). Proposing to manually maintain list of eligible labels (in this case hacktoberfest) because showing all available labels & use it similarly like language filter might make it look very crowded IMO

Sampled labels from some repos linked below

How Has This Been Tested?

Added unit tests for invalid topics usage in route

Screenshots (if appropriate):

navbar view
image

signed in user
image

unsigned in user
image

Sampled labels from some repos
image

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • [ x] My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • [ x] I have read the CONTRIBUTING document.
  • [ x] I have added tests to cover my changes.
  • All new and existing tests passed.

@schneems
Copy link
Member

Thanks for working on this! I'm going to merge in and then clean up some style preferences.

@schneems schneems merged commit e164a72 into codetriage:main Oct 13, 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.

2 participants