Skip to content

Conversation

@mrshll1001
Copy link
Contributor

Implements requested changes from https://opendataservices.plan.io/issues/60080#note-4

Changes existed across three files consisting of the "hero" banner and some paragraph copy.

In the base template, there existed some stepwise conditional logic which only replaced one word "Quality" or "Submission" depending on a template variable resulting in e.g. "Data Quality Tool" or "Data Submission Tool".

The new options are "Data Quality Checker" and "Data Submission Form", so I've moved the word "Data" inside each branch of the conditional as I think having this more readable is more important than saving a few bytes of space. There is only one word shared between the two branches of the conditional, and it'll make it easier to parse and change these in the future if they ever get re-labelled again.

Implements requested changes from https://opendataservices.plan.io/issues/60080#note-4

Changes existed across three files consisting of the "hero" banner and
some paragraph copy.

In the base template, there existed some stepwise conditional logic
which only replaced one word "Quality" or "Submission" depending on a
template variable resulting in e.g. "Data Quality Tool" or "Data
Submission Tool".

The new options are "Data Quality Checker" and "Data Submission Form",
so I've moved the word "Data" inside each branch of the conditional as I
think having this more readable is more important than saving a few
bytes of space. There is only one word shared between the two branches
of the conditional, and it'll make it easier to parse and change these
in the future if they ever get re-labelled again.
@mrshll1001
Copy link
Contributor Author

@R2ZER0 I wasn't sure which branch to merge into. I assume it's main, but please let me know if I need to edit the target of the PR

@coveralls
Copy link

Pull Request Test Coverage Report for Build 19958043103

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 80.769%

Totals Coverage Status
Change from base Build 19709940047: 0.0%
Covered Lines: 63
Relevant Lines: 78

💛 - Coveralls

Copy link

@mariongalley mariongalley left a comment

Choose a reason for hiding this comment

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

Lgtm!

@mrshll1001 mrshll1001 merged commit 09074a0 into main Dec 5, 2025
4 of 6 checks passed
@mrshll1001 mrshll1001 deleted the mm-replace-copy branch December 5, 2025 10:57
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.

5 participants