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

Criteria counts as table #20

Merged
merged 11 commits into from
Jul 20, 2023
Merged

Criteria counts as table #20

merged 11 commits into from
Jul 20, 2023

Conversation

dmundra
Copy link
Collaborator

@dmundra dmundra commented Jul 14, 2023

Fixing CivicActions/openacr-editor#23 in the CLI first. This took me a while as it was more complicated to count values into a table.

Testing steps:

  1. In the code changes you can see the previous summary being replaced with a table. Best way to view the new table summary is in HTML examples like https://civicactions.github.io/openacr/drupal-10-16-simple.html or https://civicactions.github.io/openacr/drupal-9.html.
  2. For 508 since there is no categories, I just set the column header to 'All'. I am not sure if that is valid. Do you have a suggestion for that?

@dmundra dmundra self-assigned this Jul 14, 2023
@dmundra dmundra requested review from mgifford and maggiewachs July 14, 2023 21:03
@dmundra dmundra marked this pull request as ready for review July 14, 2023 21:06
Copy link
Collaborator

@mgifford mgifford left a comment

Choose a reason for hiding this comment

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

Looks good to me. Amazing how many changes were needed to get the math to add up.

@dmundra dmundra merged commit cc7276f into main Jul 20, 2023
@dmundra dmundra deleted the criteria-counts-as-table branch July 20, 2023 15:14
dmundra added a commit that referenced this pull request Mar 12, 2024
…SA#354)

* Drupal upgrade NextGen (#18)

* Just the new drupal files

* delete old acr

* delete old acr

* pre-commit fixes

---------

Co-authored-by: pre-commit <[email protected]>

* Updated html (#19)

* Update drupal-10-16.yaml

Changing the path to CivicActions.

* Update drupal-10-16.html

Using the CLI

* Moving the 16 Simple

* Adding Drupal 10 16 Simple

* pre-commit fixes

* Updated Drupal 10 yaml version.

---------

Co-authored-by: pre-commit <[email protected]>
Co-authored-by: Daniel Mundra <[email protected]>

* Criteria counts as table (#20)

* Redoing progressPerChapter to output counts as a table.

* Removed skipping of none label.

* Adjusted table counts per row display.

* Added functions to get level and component labels to use in the progress table.

* Updating markdown examples after testing. Reset output functions.

* Changed any column header to all and added margin to simple table.

* Removed un-needed conditional check.

* Added check for undefined and updated all examples.

* Added test when first criteria has missing components. Updated output logic and regenerated examples.

* Updated tests.

* Updated version and ran JS build.

---------

Co-authored-by: Mike Gifford <[email protected]>
Co-authored-by: pre-commit <[email protected]>
dmundra added a commit that referenced this pull request Mar 12, 2024
* Redoing progressPerChapter to output counts as a table.

* Removed skipping of none label.

* Adjusted table counts per row display.

* Added functions to get level and component labels to use in the progress table.

* Updating markdown examples after testing. Reset output functions.

* Changed any column header to all and added margin to simple table.

* Removed un-needed conditional check.

* Added check for undefined and updated all examples.

* Added test when first criteria has missing components. Updated output logic and regenerated examples.

* Updated tests.

* Updated version and ran JS build.
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.

2 participants