Conversation
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
spwoodcock
left a comment
There was a problem hiding this comment.
Thanks for picking these up! 🙏
Would love if a new screenshot could be added & the coverage badge fixed, but this could be a temp solution (we would need to make follow up issues to ensure it gets done though)
|
Happy to help! I'll look into fixing the coverage badge and providing a fresh screenshot. To make sure it's exactly what you need: what specific part would you like a screenshot of? I'll update the PR as soon as I have the right image and find the new home for the coverage data. |
|
I know why the coverage is broken actually. We used to publish the docs via a github workflow (from hotosm/gh-workflows) with the We need to copy across the coverage generation logic as part of the Justfile, so we get coverage.html on the docs site, plus the coverage badge built again. As for the screenshot, it could be good to have 3 mobile screenshots combined, to make up the full readme width:
I know this is a bit of hassle, but I really feel it would help to better demonstrate the app capabilities 😄 |
|
Fixed the backend coverage logic in |
What type of PR is this? (check all applicable)
Describe this PR
This PR improves the README by fixing broken links and hiding legacy assets that are currently returning 404 errors. Specifically:
Updated the overview link to point to the correct, working URL.
Commented out the splash image and the coverage badge to prevent broken icons on the main page. Once the source files or documentation pages are back online, they can be easily restored.
AI Tool Usage
Alternative Approaches Considered
No
Review Guide
Notes for the reviewer. How to test this change?
Checklist
[optional] What gif best describes this PR or how it makes you feel?