Skip to content

rewrite github related code#738

Merged
onerandomusername merged 65 commits into
mainfrom
qt/github-rewrite
Nov 19, 2025
Merged

rewrite github related code#738
onerandomusername merged 65 commits into
mainfrom
qt/github-rewrite

Conversation

@onerandomusername

@onerandomusername onerandomusername commented Nov 1, 2025

Copy link
Copy Markdown
Owner

Summary

Rewrite the entirety of GitHub issue/pull handling code.

closes GH-571
closes GH-320
closes GH-564
closes GH-564

Key refactors

  • use a renderer system for each type of resource having a class based renderer, rather than a whole bunch of Ifs
  • centralized configuration for all behavioral changes by any configuration or feature set
  • standardized behavior
  • removal of the show more/less button, replaced with the show more button which shows a modal rather than more contents
  • smaller limits on the size of the embed
  • proper mixing of different types, such as a link and shorthand in the same message
  • properly follow and display source repository when an issue/discussion has been transferred to a different repository
  • mirror the emoji colors to the embed colour
  • when displaying comments, use the same coloured embeds that Discord webhooks use.
  • provide the correct html_url for mannequin users
  • provide the correct username/login for mannequin users on comment embeds

@onerandomusername
onerandomusername marked this pull request as draft November 1, 2025 09:03
@onerandomusername

Copy link
Copy Markdown
Owner Author

Hashing issue with ghretos classes. Need to reconstruct all of them to hash with the variables used in the url.

@onerandomusername
onerandomusername force-pushed the qt/custom-events branch 2 times, most recently from 7f85ee4 to 69541b7 Compare November 2, 2025 05:19
Base automatically changed from qt/custom-events to main November 2, 2025 05:22
onerandomusername and others added 28 commits November 19, 2025 15:00
Co-authored-by: vi <8530778+shiftinv@users.noreply.github.com>
Signed-off-by: Zenith <me@arielle.codes>
…name

Signed-off-by: Zenith <me@arielle.codes>
Signed-off-by: z <me@arielle.codes>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants