-
Notifications
You must be signed in to change notification settings - Fork 1
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.
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
12 show banner on dashboard startup when ohsome api is down #38
12 show banner on dashboard startup when ohsome api is down #38
Conversation
preparation to upgrade angular from 16 to 17
add alt-attributes to img Elements
anyway start the application, show a temporary unavailability message and the announcement banner Refs: #12
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR. I have made two smaller remarks in the comments.
Two more remarks:
- CHANGELOG entry is missing
ohsome quality analyst
is used in a lot of (publicly visible) places. This should be changed toohsome quality API
. Maybe in another PR.
@@ -49,15 +49,15 @@ <h3 class="header">Technology</h3> | |||
</p> | |||
<h4 class="header">OSM Quality Analyses are powered by:</h4> | |||
<div class=""><a href="https://oqt.ohsome.org" target="_blank"> | |||
<img class="ui image" src="assets/images/ohsome-quality-analyst_without_fonts.svg"> | |||
<img class="ui image" src="assets/images/ohsome-quality-analyst_without_fonts.svg" alt="ohsome quality analyst logo"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<img class="ui image" src="assets/images/ohsome-quality-analyst_without_fonts.svg" alt="ohsome quality analyst logo"> | |
<img class="ui image" src="assets/images/ohsome-quality-analyst_without_fonts.svg" alt="ohsome quality API logo"> |
ideally change the svg name as well
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@joker234 what about the Logo itself at the bottom: it shows "ohsome quality analyst". Should we remove it and only write ohsome quality API?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same issue as GIScience/sketch-map-tool#446
not relevant for this PR
Quality Gate failedFailed conditions |
The implemented solution handles the ohsome api downtime now similar to the oqapi downtimes.
In the OSM Hsitory Stats Tab a message about temporary unavailability will appear.
If the ohsome API is down both tabs " OSM Hsitory Stats" and "OSM Quality Analysis" will be disabled.
If there is an announcement from statuscake available it will be displayed additionally to give futher info e.g. about maintenance times.
Besides the issue angular was updated von v16 to v17 and other dependencies as well.