Skip to content

RE: Added language as a filter to explore/repo_search #29044

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 37 commits into
base: main
Choose a base branch
from
Open

RE: Added language as a filter to explore/repo_search #29044

wants to merge 37 commits into from
+389 −21

Conversation

snematoda
Copy link

continuation of #24212

snematoda and others added 30 commits August 2, 2023 19:33
@snematoda
Added language as a filter to explore/repo_search
9e7ecdd
@snematoda
Display only relevent languages
94d2db8 
Thank you @JakobDev
@snematoda
Rename Languages to ProgramLanguages
bd8e749
@snematoda
Support searching and fix error msg
d199a13
@snematoda
Switch Sort and set max height
d5f5ef2
@snematoda
Minor style fixes
d5eaca8
@snematoda
improve styling
9a77fb8
@silverwind
Update templates/explore/repo_search.tmpl
453b677
@snematoda
Merge remote-tracking branch 'up/main' into patch-1
55884cc
@snematoda
use unique css classes
120c830
@silverwind
Merge branch 'main' into patch-1
093a267
@silverwind
use flex instead of float
1e59ce2
@silverwind
move language before sort as we are no longer using float
84dd029
@silverwind
remove pointless margin, use gap
ab87133
@wxiaoguang
remove unused classes, align language names
e95e6bf
@snematoda
add support for user/org repos
5a9d382
@snematoda
Merge branch 'main' into patch-1
7c4b55d
@snematoda
use translated all instead of 'Any'
22262e0
@snematoda
display only relevent user repos
f2c304e
@snematoda
add api for languages
e50b52c
@snematoda
Merge branch 'main' into patch-1
97a73e5
@silverwind
Update routers/api/v1/repo/languages.go
85f1389
@snematoda
added ownerID to languages api
1393328 
- wrote api tests
@snematoda
Merge branch 'main' into patch-1
219d4d4
@silverwind
Merge branch 'main' into patch-1
3794e37
@snematoda
Improved api and db calls
2a8d0c8
@snematoda
add tests and fix styling
bc4d8b5
@snematoda
move styles to base.css
00fb639
@snematoda
Merge branch 'main' into patch-1
3ae0bb8
@snematoda
hotfixes
7fb0de1 
+ fix for private repo being shown
+ css fix
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Feb 4, 2024
@github-actions github-actions bot added modifies/api This PR adds API routes or modifies them modifies/frontend labels Feb 4, 2024
KN4CK3R
KN4CK3R reviewed Feb 5, 2024
View reviewed changes
@yp05327
Copy link
Contributor

yp05327 commented Feb 9, 2024

Although the original push is 2023, but should we change the copyright from 2023 to 2024? 😄

@snematoda
Copy link
Author

Although the original push is 2023, but should we change the copyright from 2023 to 2024? 😄

Not too sure about this ... But, Isn't it generally preferred to use the date the code was pushed over the date of release?

@yp05327 yp05327 mentioned this pull request Feb 22, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lunny lunny lunny left review comments

@lafriks lafriks lafriks left review comments

@KN4CK3R KN4CK3R KN4CK3R left review comments

@wxiaoguang wxiaoguang wxiaoguang left review comments

Assignees
No one assigned
Labels
lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. modifies/api This PR adds API routes or modifies them
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
9 participants
@snematoda @yp05327 @lunny @lafriks @KN4CK3R @wxiaoguang @silverwind @techknowlogick @GiteaBot