Add descriptions for all installable apps - #24
Open
MetzinAround wants to merge 3 commits into
Open
MetzinAround wants to merge 3 commits into
MetzinAround wants to merge 3 commits into
Conversation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: e6afc5b4-5772-4b44-90be-268a218b93fd
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: e6afc5b4-5772-4b44-90be-268a218b93fd
MetzinAround
requested review from
avni,
crnicholson and
martinwoodward
and
a lite review from Copilot
September 13, 2026 19:23
MetzinAround
requested review from
crnicholson
and removed request for
avni,
crnicholson and
martinwoodward
September 13, 2026 19:26
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
Unresolved user-facing copy issues remain in src/content/app-descriptions.ts.
Get a fresh assessment by requesting another Copilot review.
Review tier: Lite
Findings: 9
Open findings (9)
What changed in this PR
Adds descriptions for external installable apps and makes them available in the /edit app store.
Changes:
- Adds descriptions for 18 external apps.
- Merges external descriptions with local app metadata.
- Documents app-description conventions.
| File | Reviewed changes |
|---|---|
src/pages/edit.astro |
Merges local and external descriptions. |
src/content/app-descriptions.ts |
Defines external app descriptions; copy corrections remain. |
.github/copilot-instructions.md |
Documents app-description and catalog-key conventions. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Author
|
@crnicholson Some of the apps didn't have a description associated, so I went through and made a 1 sentence describer for each one. Copilot update instructions that each app should have a description so when this page populates, the description comes with it. This needs to go into the app descriptions PR first before that one is merged. |
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds descriptions for the installable apps that are not part of the local content collection.
The edit page now merges these descriptions with the existing app metadata so every app in the badger/home catalog can show a description.
Build verified with
npm run build.Targets #23.