Draft: Flyout: Build status indicator #446
Conversation
Do a very basic build status indictor, which shows the status of the current/last build.
|
Don't stop exploring addons and JS, but I do think we should discuss this feature first. This will show failed build state to all reader users. That seems bad for large projects and commercial users, where reader users most likely don't know what Read the Docs or a failed build is. A failed state, red circle, or describing the build as failed all seem like things we should only throw at maintainers. I'd say the flyout is the wrong place to surface this, and this would make more sense in a dedicated PR build UI. This might be the same UI as diffing too. |
I agree on this. We touched on this idea in a previous issue but we found we need to do something else regarding caching: #88
We talked about maybe adding an indicator on the notification. I think we are entering on the phase where we would get a lot of benefit from discussing "what are the next steps regarding addons UI" as soon as possible. We all want to implement more addons features, but nobody feels comfortable about where to put them. Building these features will be pretty quick once we define that. |
This is useful context. I see we already noted in that issue that we'll want to hit the API directly for this data to avoid caching, which makes a lot of sense 👍 |
Do a very basic build status indictor,
which shows the status of the current/last build.
I couldn't get it to center vertically for some reason,
but just wanted to throw something up as a starting point.
Example