4
9
Fork
You've already forked PipePipeClient
7

[New Features] [BiliBili] extract and display stream detailed statistic & comment pictures dialog & merge upstream (Client Side) #11

Closed
chr_56 wants to merge 16 commits from chr_56:client_info_extension into dev
pull from: chr_56:client_info_extension
merge into: NullPointerException:dev
NullPointerException:dev
NullPointerException:legacy
NullPointerException:main
Contributor
Copy link

This contains:

  1. Merge upstream changes on Description fragments with relative utilities update:
    Upstream pulled up a BaseDescriptionFragment and use it in Channel's about tab, and introduced
    a model class Image in extractor to improve avatars and thumbnails extracting (all image url
    extracting method are replaced with corresponding Image collection extracting method, but I
    kept the legacy codes and redirected to the legacy since we can not afford large changes on
    "unmaintained" services). This as well results in supporting various size images extracting
    instead of one url in Metadata section (by refactoring of TextLinkifier with new text-span
    utils).
  2. Extract and display BiliBili stream detailed statistic:
    Some "very important" BiliBili stream stats (for "most BiliBili user"), like coins count and
    danmaku count and view count, are showed in Metadata section (We can later display them along
    with "YouTube like count" position if available) by extending StreamInfo with relative changes
    on DescriptionFragment.
  3. Add PictureDialog to view pictures in comments:
    I extended CommentsInfoItem to store attached pictures in formats of Image, and
    created PictureDialog to view pictures in CommentsInfoItemHolder. Currently, it's in state
    of "just working" requiring ongoing refinement and polish.

This require correspond Extractor PR.

This contains: 1. Merge upstream changes on Description fragments with relative utilities update: Upstream pulled up a `BaseDescriptionFragment` and use it in Channel's about tab, and introduced a model class `Image` in extractor to improve avatars and thumbnails extracting (all image url extracting method are replaced with corresponding `Image` collection extracting method, but I kept the legacy codes and redirected to the legacy since we can not afford large changes on "unmaintained" services). This as well results in supporting various size images extracting instead of one url in Metadata section (by refactoring of `TextLinkifier` with new text-span utils). 2. Extract and display BiliBili stream detailed statistic: Some "very important" BiliBili stream stats (for "most BiliBili user"), like coins count and danmaku count and view count, are showed in Metadata section (We can later display them along with "YouTube like count" position if available) by extending `StreamInfo` with relative changes on DescriptionFragment. 3. Add PictureDialog to view pictures in comments: I extended `CommentsInfoItem` to store attached pictures in formats of `Image`, and created `PictureDialog` to view pictures in `CommentsInfoItemHolder`. Currently, it's in state of "just working" requiring ongoing refinement and polish. This require correspond [Extractor PR](https://codeberg.org/NullPointerException/PipePipeExtractor/pulls/8).

Please make separate PR for the 2 features.

Please make separate PR for the 2 features.

Merged locally with some adjustments. Thanks.
btw, do you still have other plans? We can discuss them before starting to avoid merge conflicts.

Merged locally with some adjustments. Thanks. btw, do you still have other plans? We can discuss them before starting to avoid merge conflicts.
Author
Contributor
Copy link

Thank you.

And, I may try to open an issue first before starting implement a new feature next times.

PS: I have reported to Codeberg community the problem that Extractor repo can not receive any PR. It is said to be caused by stale lock files and would notify as soon it recovered.

Thank you. And, I may try to _open an issue first_ before starting implement a new feature next times. PS: I have reported to Codeberg community the problem that Extractor repo can not receive any PR. It is said to be caused by stale lock files and would notify as soon it recovered.

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
NullPointerException/PipePipeClient!11
Reference in a new issue
NullPointerException/PipePipeClient
No description provided.
Delete branch "chr_56:client_info_extension"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?