Skip to content
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

removed the installed state #256

Merged
merged 1 commit into from
Oct 15, 2021

Conversation

oliver-sanders
Copy link
Member

@oliver-sanders oliver-sanders commented Oct 11, 2021

PR trio: cylc/cylc-flow#4459, #256, cylc/cylc-ui#805

  • The installed state was the same as "stopped" but with the additional
    information that the workflow had not yet been run.
  • Added this information to the workflow status message in cylc-uiserver
    (as it is offline information).
  • Improved the workflow status message provided by cylc-flow.
  • Put the status message into the UI toolbar.
    workflow-status-message

Requirements check-list

  • I have read CONTRIBUTING.md and added my name as a Code Contributor.
  • Contains logically grouped changes (else tidy your branch by rebase).
  • Does not contain off-topic changes (use other PRs for other changes).
  • Appropriate tests are included (unit and/or functional).
  • No change log entry required (why? e.g. invisible to users).
  • No documentation update required.
  • No dependency changes.

@codecov-commenter
Copy link

Codecov Report

Merging #256 (5ce2817) into master (3815aaa) will increase coverage by 0.06%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #256      +/-   ##
==========================================
+ Coverage   75.00%   75.06%   +0.06%     
==========================================
  Files          11       11              
  Lines         736      738       +2     
  Branches      125      126       +1     
==========================================
+ Hits          552      554       +2     
  Misses        158      158              
  Partials       26       26              
Impacted Files Coverage Δ
cylc/uiserver/data_store_mgr.py 65.93% <100.00%> (+0.37%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3815aaa...5ce2817. Read the comment docs.

@oliver-sanders oliver-sanders marked this pull request as ready for review October 11, 2021 12:13
@oliver-sanders
Copy link
Member Author

(mypy failing due to comparing against cylc-flow master branch)

@oliver-sanders
Copy link
Member Author

  • [flow] Fixed a bug in the data store where the workflow status wasn't updated to stopping (unless something else triggered an update).
  • [uis] Added logic to change the status from * to stopped on receiving the shutdown message.
  • [ui] Added the status message to the gscan workflow icon tooltips.

opened: #257

* The installed state was the same as "stopped" but with the additional
  information that the workflow had not yet been run.
* Added this information to the workflow status message in cylc-uiserver
  (as it is offline information).
* Improved the workflow status message provided by cylc-flow.
* Put the status message into the UI toolbar.
Copy link
Member

@hjoliver hjoliver left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One review will do, gotta make this compatible with cylc-flow now (hopefully I'm not being too hasty!)

@hjoliver hjoliver merged commit b4f6711 into cylc:master Oct 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants