Skip to content

tidy: docs and some tidying #8614

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
Merged
merged 20 commits into from
Oct 14, 2025
Merged

tidy: docs and some tidying #8614

merged 20 commits into from
Oct 14, 2025

Conversation

psychedelicious
Copy link
Collaborator

Summary

  • Improved docs for some features
  • Tidy some unused code paths

Related Issues / Discussions

n/a

QA Instructions

n/a

Merge Plan

n/a

Checklist

  • The PR has a short but descriptive title, suitable for a changelog
  • Tests added / updated (if applicable)
  • ❗Changes to a redux slice have a corresponding migration
  • Documentation added / updated (if applicable)
  • Updated What's New copy (if doing a release after this PR)

@github-actions github-actions bot added api python PRs that change python files Root invocations PRs that change invocations backend PRs that change backend files services PRs that change app services frontend-deps PRs that change frontend dependencies frontend PRs that change frontend files python-tests PRs that change python tests labels Oct 14, 2025
@psychedelicious
tidy: removing unused code paths 1
dae2b80
@psychedelicious
tidy: removing unused code paths 2
1958cb5
@psychedelicious
tidy: removing unused code paths 3
b3594c5
@psychedelicious
tidy: removing unused code paths 4
5968d71
@psychedelicious
tidy: removing unused code paths 5
ea71f1d
@psychedelicious
tidy: removing unused code paths 6
2810c47
@psychedelicious
tidy: removing unused code paths 7
373718f
@psychedelicious
tidy(ui): lift error boundary reset cb outside component
35f5425
@psychedelicious
docs(ui): add comments for nes
2ba8a98
@psychedelicious
docs(ui): add comments for image context menu
2a4babf
@psychedelicious
tidy(ui): rename GalleryPanel file
709c9e7
@psychedelicious
tidy(ui): rename GalleryImageGrid and add comments
8f08051
@psychedelicious
docs(ui): add high-level readmes for various features
3eec5da
@psychedelicious
tidy(ui): clean up useSyncQueueStatus
267c24c
@psychedelicious
tidy: app "config", settings modal, infill methods
2c632f1 
We had an "infill methods" route that long ago told the frontend infill
method, upscale method (model), NSFW checker, and watermark feature
availability.

None of these were used except for the patchmatch check. Removed them,
made the check exclusively for patchmatch, updated related code in redux
app startup listeners and settings modal.
@psychedelicious psychedelicious force-pushed the psyche/tidy/unused-codepaths branch from 688dc4c to dfc3f4e Compare October 14, 2025 23:23
@psychedelicious psychedelicious enabled auto-merge (rebase) October 14, 2025 23:23
@psychedelicious psychedelicious merged commit d988891 into main Oct 14, 2025
13 checks passed
@psychedelicious psychedelicious deleted the psyche/tidy/unused-codepaths branch October 14, 2025 23:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hipsterusername hipsterusername hipsterusername approved these changes

@blessedcoolant blessedcoolant Awaiting requested review from blessedcoolant blessedcoolant is a code owner

@maryhipp maryhipp Awaiting requested review from maryhipp maryhipp is a code owner

@lstein lstein Awaiting requested review from lstein lstein is a code owner

@ebr ebr Awaiting requested review from ebr

Assignees

No one assigned

Labels

api backend PRs that change backend files frontend PRs that change frontend files frontend-deps PRs that change frontend dependencies invocations PRs that change invocations python PRs that change python files python-tests PRs that change python tests Root services PRs that change app services

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@psychedelicious @hipsterusername