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

Add data fields to service_page #474

Merged
merged 6 commits into from Oct 6, 2020
Merged

Add data fields to service_page #474

merged 6 commits into from Oct 6, 2020

Conversation

weitzman
Copy link
Contributor

@weitzman weitzman commented Sep 23, 2020

Description:
Explain the technical implementation of the work done.

Jira: (Skip unless you are MA staff)
https://jira.mass.gov/browse/DP-19887

To Test:

  • Populate the new fields (verify form placement, help text, and conditionals) on a service_page and assure that the metatags match the spec in Jira.

Screenshots/GIFs:
Use something like licecap to capture gifs to demonstrate behaviors.


Peer Review Checklist

@weitzman weitzman marked this pull request as ready for review September 25, 2020 17:19
@tom-fleming
Copy link
Contributor

LGTM!

@tom-fleming tom-fleming merged commit d791d60 into develop Oct 6, 2020
@tom-fleming tom-fleming deleted the serviceData branch October 6, 2020 18:14
tom-fleming added a commit that referenced this pull request Oct 20, 2020
* config change and changelog (#479)

* DP 20094 help text change for data resource type field (#480)

* help text changes

* changelog

* simple text change to trigger rebuild

* changelog update and remove old changelog files

* Add data fields to service_page (#474)

* Add data fields to service_page

* Use related terms

* Add support for service_page in mass_related_terms token.

* add field to be used for description

* conditional logic and form layout changes

* Fix coding standards.

Co-authored-by: Joe Galluccio <joseph.galluccio@state.ma.us>

* Remove balancer in Terraform (#481)

* Remove balancer IP address.

* Add changelog

* try uppercase

* Declare the variable again. Add logging

* Remove super verbose logging.

* Upper

* Dp 20172 promo button text size (#486)

* config changes

* changelog

* Add changelog (#485)

* DP-19747 - Reduce log noise (#484)

* Remove debug loggings.

* Add changelog

* CS

* Fix doc comment.

* bool

* Update basic iframe URL so that node ID is correctly passed to Supers… (#483)

* Update basic iframe URL so that node ID is correctly passed to Superset filter.

* changelog.

* fixing syntax on changelog

Co-authored-by: joeg8612 <joseph.galluccio@state.ma.us>

* DP-19787: Fix the needs attention views (#476)

* DP-19787: Fix the needs attention views

The calls to superset were calling a non-existant old view.
In addition, the superset API has changed and this decreases
the batch size from 1000 to 500 to eliminate timeouts.

* DP-19787: Add changelog

* DP-19787: Increase the cron timeout for superset queue items

* DP-19787: Restart mass_superset queue

* Added changelog - DP-19120

Co-authored-by: joeg8612 <joseph.galluccio@state.ma.us>
Co-authored-by: CircleCI Deployment Bot <circleci@example.com>
Co-authored-by: Moshe Weitzman <weitzman@tejasa.com>
Co-authored-by: Yukiko Gannett <yukikogannett@gmail.com>
Co-authored-by: adamcogbill <33554893+adamcogbill@users.noreply.github.com>
Co-authored-by: Robert Stephens <roberttstephens@gmail.com>
ygannett added a commit that referenced this pull request Oct 20, 2020
* DP-19120: header navigation changes

* Template changes

* Adding changelog

* Adding default conf

* phpcs issue fix

* adjust theming templates for additional changes done in Mayflower

* DP-19328 Hamburgermenu for binder page (#406)

* update the <head> content generation logic in /themes/custom/mass_theme/templates/layout/page--node--without-main.html.twig for binder page

* add changelog for DP-19328

* remove commented out code

* update the menu label change in Mayflower

* match MF change in theming

* DP-19876: Contextual login links are missing in branch with new global nav (#468)

* DP-19876: Contextual links fix

* DP-19876: Added changelog

Co-authored-by: Tom Fleming <tom@lastcallmedia.com>

* DP-20203 Missing alert overlay in info details page (#490)

* add alert overlay container to page--node--without-main.html.twig

* add changelog for DP-20203

* Added back header_mixed_urls setting - DP-19120

* Resolve DP-19120 merge conflicts (#492)

* config change and changelog (#479)

* DP 20094 help text change for data resource type field (#480)

* help text changes

* changelog

* simple text change to trigger rebuild

* changelog update and remove old changelog files

* Add data fields to service_page (#474)

* Add data fields to service_page

* Use related terms

* Add support for service_page in mass_related_terms token.

* add field to be used for description

* conditional logic and form layout changes

* Fix coding standards.

Co-authored-by: Joe Galluccio <joseph.galluccio@state.ma.us>

* Remove balancer in Terraform (#481)

* Remove balancer IP address.

* Add changelog

* try uppercase

* Declare the variable again. Add logging

* Remove super verbose logging.

* Upper

* Dp 20172 promo button text size (#486)

* config changes

* changelog

* Add changelog (#485)

* DP-19747 - Reduce log noise (#484)

* Remove debug loggings.

* Add changelog

* CS

* Fix doc comment.

* bool

* Update basic iframe URL so that node ID is correctly passed to Supers… (#483)

* Update basic iframe URL so that node ID is correctly passed to Superset filter.

* changelog.

* fixing syntax on changelog

Co-authored-by: joeg8612 <joseph.galluccio@state.ma.us>

* DP-19787: Fix the needs attention views (#476)

* DP-19787: Fix the needs attention views

The calls to superset were calling a non-existant old view.
In addition, the superset API has changed and this decreases
the batch size from 1000 to 500 to eliminate timeouts.

* DP-19787: Add changelog

* DP-19787: Increase the cron timeout for superset queue items

* DP-19787: Restart mass_superset queue

* Added changelog - DP-19120

Co-authored-by: joeg8612 <joseph.galluccio@state.ma.us>
Co-authored-by: CircleCI Deployment Bot <circleci@example.com>
Co-authored-by: Moshe Weitzman <weitzman@tejasa.com>
Co-authored-by: Yukiko Gannett <yukikogannett@gmail.com>
Co-authored-by: adamcogbill <33554893+adamcogbill@users.noreply.github.com>
Co-authored-by: Robert Stephens <roberttstephens@gmail.com>

Co-authored-by: Yukiko Gannett <yukikogannett@gmail.com>
Co-authored-by: Tom Fleming <tom@lastcallmedia.com>
Co-authored-by: joeg8612 <joseph.galluccio@state.ma.us>
Co-authored-by: CircleCI Deployment Bot <circleci@example.com>
Co-authored-by: Moshe Weitzman <weitzman@tejasa.com>
Co-authored-by: adamcogbill <33554893+adamcogbill@users.noreply.github.com>
Co-authored-by: Robert Stephens <roberttstephens@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants