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

IV: Show channel handle in search results #3791

Merged

Conversation

ChunkyProgrammer
Copy link
Member

@ChunkyProgrammer ChunkyProgrammer commented Jul 18, 2023

IV: Show channel handle in search results

Pull Request Type

  • Feature Implementation

Related issue

Video Count for channels is almost always 0 (it might always be 0)

Description

Display the channel handle when it exists + fix missing translations for ft-list-channel

Testing

  • Use Invidious API
  • Search 'MrBeast'
  • See channel handle instead of video count for the Mr Beast channel

Desktop

  • OS: Windows
  • OS Version: 10
  • FreeTube version: 0.18.0

Additional context

Relies on iv-org/invidious#3994 to be merged

@github-actions github-actions bot added the PR: waiting for review For PRs that are complete, tested, and ready for review label Jul 18, 2023
@FreeTubeBot FreeTubeBot enabled auto-merge (squash) July 18, 2023 16:07
@ChunkyProgrammer ChunkyProgrammer removed the PR: waiting for review For PRs that are complete, tested, and ready for review label Jul 18, 2023
@absidue
Copy link
Member

absidue commented Jul 18, 2023

The test cases in this pull request should be tested too: #3099

@github-actions
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions
Copy link
Contributor

Conflicts have been resolved. A maintainer will review the pull request shortly.

@efb4f5ff-1298-471a-8973-3d47447115dc

Upstream PR finally merged letssgoooo!

@ChunkyProgrammer ChunkyProgrammer added PR: waiting for review For PRs that are complete, tested, and ready for review and removed PR: WIP labels Oct 7, 2023
@efb4f5ff-1298-471a-8973-3d47447115dc
Copy link
Member

efb4f5ff-1298-471a-8973-3d47447115dc commented Oct 7, 2023

If possible i would like it to be formatted this way
Capture3
Capture3

We do formatting like this everywhere but not here so it would be nice if that also could be addressed in here

So Include • this dot

Would also be nice to put handle first like YT

Copy link
Collaborator

@PikachuEXE PikachuEXE left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested via searching LTT
(One capped with local one with IV)
Screenshot 2023-10-10 at 13 14 49
Screenshot 2023-10-10 at 13 16 32

Copy link
Member

@absidue absidue left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works with all cases I tested. Also tested both local and Invidious API, just to make sure it still works for the local API (cleared the serach cache in between).

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@FreeTubeBot FreeTubeBot merged commit fffd139 into FreeTubeApp:development Oct 10, 2023
5 checks passed
@github-actions github-actions bot removed the PR: waiting for review For PRs that are complete, tested, and ready for review label Oct 10, 2023
PikachuEXE added a commit to PikachuEXE/FreeTube that referenced this pull request Oct 11, 2023
* development:
  IV: Show channel handle in search results (FreeTubeApp#3791)
  Bump electron from 22.3.25 to 22.3.26 (FreeTubeApp#4120)
  Bump lefthook from 1.5.0 to 1.5.2 (FreeTubeApp#4122)
  Bump the eslint group with 4 updates (FreeTubeApp#4119)
  Bump sass from 1.68.0 to 1.69.0 (FreeTubeApp#4121)
  Bump marked from 9.0.3 to 9.1.0 (FreeTubeApp#4123)
  Bump version number to v0.19.1
  Fix styling for Channel page on desktop view (FreeTubeApp#4112)
  Set new consent cookie and supress tracking cookies on the watch page (FreeTubeApp#4013)
  Translated using Weblate (Italian)
  Update translation files
  Added translation using Weblate (Belarusian)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants