Skip to content

fix: WEBRIP type#1255

Merged
Audionut merged 1 commit intoAudionut:masterfrom
tdrkmn:fix/webrip-type
Feb 18, 2026
Merged

fix: WEBRIP type#1255
Audionut merged 1 commit intoAudionut:masterfrom
tdrkmn:fix/webrip-type

Conversation

@tdrkmn
Copy link

@tdrkmn tdrkmn commented Feb 18, 2026

Summary by CodeRabbit

  • Bug Fixes
    • Updated ShareIsland tracker's content type identification for improved accuracy.

@github-actions
Copy link

Thanks for taking the time to contribute to this project. Upload Assistant is currently in a complete rewrite, and no new development is being conducted on this python source at this time.

If you have come this far, please feel free to leave open, any pull requests regarding new sites being added to the source, as these can serve as the baseline for later conversion.

If your pull request relates to a critical bug, this will be addressed in this code base, and a new release published as needed.

If your pull request only addresses a quite minor bug, it is not likely to be addressed in this code base.

Details for the new code base will follow at a later date.

@coderabbitai
Copy link

coderabbitai bot commented Feb 18, 2026

No actionable comments were generated in the recent review. 🎉


📝 Walkthrough

Walkthrough

Updated the WEBRIP type-to-ID mapping in ShareIsland tracker from "15" to "51" in the get_type_id method. No control flow or error handling changes. All other type mappings remain unchanged.

Changes

Cohort / File(s) Summary
Type ID Mapping Update
src/trackers/SHRI.py
Updated WEBRIP type ID from "15" to "51" in the type-to-ID mapping dictionary.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A number hops from fifteen's grace,
To fifty-one, its rightful place,
WEBRIP finds its true ID home,
Through ShareIsland's type-ID tome! 🌟

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'fix: WEBRIP type' directly describes the main change - updating the WEBRIP type-to-ID mapping in the SHRI tracker from '15' to '51'. It is specific and accurately reflects the changeset.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@Audionut Audionut merged commit 0991595 into Audionut:master Feb 18, 2026
2 checks passed
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.

2 participants