-
Notifications
You must be signed in to change notification settings - Fork 8
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
Copy tech alerts over to relevant product pages so they can be found #376
base: main
Are you sure you want to change the base?
Conversation
Preview is ready!
You can also view the deploy log or read the documentation. |
I really like this so far. I'll just ask some questions to see how your design will handle these cases:
Also, Im not sure if this heading should be 'Known issues': https://pr-376-preview.khpreview.dea.ga.gov.au/data/product/dea-hotspots/?tab=quality#dea-hotspots-tech-alerts |
I did some quick checks while I was preparing this PR. It seems that where there was actual useful information (as opposed to just a note that a new version was available), the information was already in the History tab. I don't think we need to post the Tech Alert in its entirety into the History tab, but we do need to make sure that any actual useful information about changes are included in the product history tab. As of this PR, anything in the Tech Alerts has been cross checked.
Temp downtimes, no, but anything long lasting that could affect quality, yes. Again, this PR has moved anything important across.
I wasn't really sure what the heading should be. I just used "Known Issues" because it seemed relevant for the content, but I am very happy to take suggestions.
|
Thanks for answering those questions. That all seems good. I thought about the 'Known issues' heading and my best suggestion is:
This is because the list may contain some issues that have been resolved (and they should be marked as resolved, as in the Tech Alerts) |
Though some of them won't have been resolved. Maybe "Known issues are resolutions" to capture both? |
We can do that. Now I'm just nitpicking - I feel that 'Known' isn't needed, since all issues are known issues. And it can be more 'skimmable' to the reader if the heading starts with the word 'Issue'. |
I really love this! A "Known issues" register is something we've been needing for a long time - this will be really, really useful for our products (I know @simonaoliver has raised this before too). Some stray thoughts:
I kind of like "Known issues" for its simplicity - it's commonly used in these kinds of data repos and shows we are aware of these problems but haven't necessarily fixed them yet. But "Known issues and alerts" could work too. |
My thought behind including key resolved issues is that some users may have downloaded or created products from older versions of the data where the resolved issues are still relevant. I dont think any of the outages are useful, but quality issues that have been resolved could be very relevant? |
Yep, fair enough! I guess we can just clearly label them as resolved, and perhaps move them to the bottom of the list of issues so they are a bit less prominent |
Hi Robbi and Claire - I think those are good ideas.
|
The syntax for the drop downs is really easy, so you and I can manage that, Benji? Maybe we can collapse the resolved ones, and keep the open ones as plain text? |
It's fairly easy - I was mainly thinking of the other stakeholders who would be encouraged to keep this up to date going forward. If we make it as easy for them as possible, they are more likely to edit it as opposed to neglecting it. |
As a stakeholder, as long as there's a nice template/working example present in the repo, I think we'll be able to work it out. 🙂 Maybe we can test it out and see what it would look like? If it doesn't seem worth the effort we can reconsider... I can just see certain listings (e.g. ARD) accumulating a rapidly growing collection of known issues as time goes on, and it'd be good keep things neat and tidy. |
OK. So to summarise to make sure I understand the consensus:
I'll go in and amend this PR with these changes so we can see how it'll look. |
I think that looks pretty good now? Thoughts? |
We will need to add this Tech Alert to the relevant product pages once the PR has been merged: #384 |
There are quite a few tech alerts that contain very pertinent information about product bugs and quality information that unless you know it exists in the tech register, is easy to miss.
I have copied the relevant information across into the "Quality" tabs of the relevant products so that all the information about a product is together in the product pages.