-
Notifications
You must be signed in to change notification settings - Fork 9
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
Add descriptions #626
Add descriptions #626
Conversation
…atus_descriptions' into feature/add_work_status_descriptions Merge with changes from remote
…g constraints with new name
… been merged with master
@ja573 I can't request you as a reviewer on your own PR, but I'd appreciate you reviewing my sections, especially my db migrations - thanks! |
@@ -75,7 +75,9 @@ pub enum WorkType { | |||
#[cfg_attr( | |||
feature = "backend", | |||
derive(DbEnum, juniper::GraphQLEnum), | |||
graphql(description = "Publication status of a work"), | |||
graphql( | |||
description = "Publication status of a work throughout its lifecycle. For a visual representation of the workflow, refer to the work status flowchart https://github.com/thoth-pub/thoth/wiki/Thoth_Works#work-status-flowchart" |
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.
FYI, one of the boxes in this flowchart isn't displaying properly for me in Firefox (although it's working in Chromium, weirdly)
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.
Which box is it? They display fine for me in firefox and safari
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.
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.
That's odd – does it display properly in the original #595 (comment) ? It should be exactly the same mermaid code
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.
Hah, yes, I'm getting the same display problem there, but I definitely didn't notice it when originally reading that discussion! Could be to do with updates I've installed in the meantime...
No description provided.