Skip to content

Add report issue button in app info, and add option for apps to require minimum firmware version - #92

Merged
gfwilliams merged 11 commits into
espruino:masterfrom
RKBoss6:fwrequire
Jul 3, 2026
Merged

Add report issue button in app info, and add option for apps to require minimum firmware version#92
gfwilliams merged 11 commits into
espruino:masterfrom
RKBoss6:fwrequire

Conversation

@RKBoss6

@RKBoss6 RKBoss6 commented Jun 28, 2026

Copy link
Copy Markdown
Contributor

Adds the report issue button inside the app info modal, talked about in #89 , which opens a link to a issue with the name prefilled in the title.

Also adds support for a metadata field, requiredFw for apps to specify a firmware version needed to run, like the loading animation app, which needs a specific firmware. This will help ensure that users don't install apps not meant for their firmware version, or uses features not supported by their firmware, and will overall improve the robustness of the app loader itself, ensuring everyone has a positive experience with all apps, instead of needing to look closer to see if their firmware version matches.

Example metadata line:
requiredFw: "2v29"

This all isn't up on my app loader, as #91 is up right now, hopefully we can get that merged and then this, but everything looks good and works properly.

Copilot AI review requested due to automatic review settings June 28, 2026 20:49

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@RKBoss6 RKBoss6 changed the title Apps can require firmware versions Add report issue button in app info, and add option for apps to require minimum firmware version Jun 29, 2026
Comment thread js/ui.js Outdated
Comment thread js/index.js Outdated
@gfwilliams

Copy link
Copy Markdown
Member

Thanks! The new field would need documenting in the README and handling in the sanity checker too. And it should probably show that information in the new app info window too.

Is this in response to an actual issue someone had though? In general if someone has an out of date firmware an orange warning pops up every time they connect, so I tend to think that's good enough. I don't think we should be wasting time trying to deal with users that insist on running out of date firmware - since they're likely to hit some issue sooner or laster.

@RKBoss6

RKBoss6 commented Jun 29, 2026

Copy link
Copy Markdown
Contributor Author

Yes actually - I don't remember what app I had tried to install, but at one point (when I first got my bangle) an app I had installed wouldn't work and loaded indefinitely, and once I had connected it to the IDE, I realized that it needed the latest cutting edge firmware to work, which wasn't mentioned in the README anywhere...
In general though, especially with the addition of apps for Bangle.js 3 using its new features, it seems like a good idea to ensure users get the best experience possible, especially because many of the firmware features will be released throughout versions...

@gfwilliams

Copy link
Copy Markdown
Member

Ok, great. Let's stick this in then.

Now I've tweaked the github issue format it'd be great if you could get &author=%40foo set if the author exists, and we're good to go

@RKBoss6

RKBoss6 commented Jun 30, 2026

Copy link
Copy Markdown
Contributor Author

All ready now!

@gfwilliams

Copy link
Copy Markdown
Member

Awesome - thanks!

@gfwilliams
gfwilliams merged commit 3d9c0fa into espruino:master Jul 3, 2026
1 check passed
gfwilliams added a commit to espruino/BangleApps that referenced this pull request Jul 3, 2026
@RKBoss6
RKBoss6 deleted the fwrequire branch August 10, 2026 20:16
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.

3 participants