Repository navigation
Update About settings and add in-app support - #78
Conversation
There was a problem hiding this comment.
Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.
An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: a9b8adb6b3
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Codecov Report❌ Patch coverage is 📢 Thoughts on this report? Let us know! |
About settings now show the installed app version, link to that release's changelog, open the client repository, and provide an in-app support form with image/video attachments. Licenses and Support use the shared page header, and the license browser disables animations.
Issue and requirements
Requested directly by the maintainer. Related design requirements: NFR-5_1 and NFR-3_4. Support submission requires PapyrusReader/server#41.
Summary
Testing
33 focused tests passed across About settings, licenses, support form, attachments, API handling, and confirmation.
tools/papyrus check client, the web build, andgit diff --checkpassed. Dark/light/e-ink and narrow layouts with larger text are covered. The local reader override points to the exact pinned reader revision and is excluded from the commit.UI evidence
The support confirmation was visually checked in a local render with the actual sidebar and shared page header. It uses the form's width, standard text sizes, and a compact settings-style issue row. No screenshot is attached. Native device acceptance remains unrun.
Compatibility and migration
No persistence migration. Requires the companion server endpoint for submissions. New runtime dependency:
package_info_plus;url_launcheris now an explicit dependency. Builds retain the current application version.Notes
Feedback and attachments are public, which the form discloses before submission. Errors retain the draft and display
Submission failed.No automatic submission retries.Checklist