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

[pull] dev from opf:dev #120

Merged
merged 31 commits into from
Feb 2, 2024
Merged

[pull] dev from opf:dev #120

merged 31 commits into from
Feb 2, 2024

Conversation

pull[bot]
Copy link

@pull pull bot commented Feb 2, 2024

See Commits and Changes for more details.


Created by pull[bot]

Can you help keep this open source service alive? 💖 Please sponsor : )

mereghost and others added 30 commits January 30, 2024 18:46
- Adds VCR options for re-recording and filtering sensitive data
- Adds error handling to Util.using_admin_token
- Improves logging of multiple commands
* Runs `bin/rails generate migration`.

* Adds column to `work_packages` and `work_package_journals`.

* Backfills column for non leaf work packages.
I prefer having this explicitly marked as false than having to infer
that because of the attribute not being listed, it's non-writable.
Gives the client knowledge of this field in order to pick up on it
and render it within forms.
Returns the derived_done_ratio in the response when fetching a work
package via the API.
Provides the localized name for this field.
With the added column to the work packages table, a new page was
rendered when exporting the PDF under test. This is a non-functional
change and simply situational due to the newly added column.

Adapted the expected output to match what is being outputted now by
the renderer.
* Instead of modifying the `done_ratio` of other work packages in the
  chain, this commit modifies the `derived_done_ratio` of all relevant
  work packages in the chain is this is now the value that is derived.

* Allows `done_ratio` to be writable as long as the setting allows it.
  There is no longer a restriction on the work package being a leaf
  to be writable.
* Runs `bundle exec rubocop -a` on spec file and corrects offenses.
* There is no scenario where this column is NULL as the column default
  is 0.
* Ensures the system user is created if it doesn't already exist.
Ensures qualitative properties of the field.
If a work package is a leaf, there is no done_ratio to derive.

Hence, we'd only like to render the derived_done_ratio if the work
package has children in its hierarchy.
Navigates to a query with a view of descendant work packages to
inspect the breakdown of child completion.
We now expect the derived_done_ratio to be updated instead of the
parent's own done_ratio.
Creating activity entries just results in noise and is not being used
at the moment as using update_all bypasses Rails hooks.
De-duplicates the generation as the links are identical for both
DisplayField modules.
Since the derived % complete is no longer included in the type form
configuration, it's no longer included in the pdf export, so the need
for adapting the spec in this PR is no longer relevant.
…one-ratio-to-work-packages

[#51188] Split "% Complete" into a derived-value and self-value
Database can return project children in any order.
@pull pull bot added the ⤵️ pull label Feb 2, 2024
@pull pull bot merged commit 16c1253 into l3dlp-sandbox:dev Feb 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants