Update basic iframe URL so that node ID is correctly passed to Supers… #483
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
joeg8612
approved these changes
Oct 13, 2020
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
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
…et filter.
Description:
This is what I believe you all refer to as a "backport config"? Anyway, I updated the route iframe configuration for "basic page dashboards" in the CMS, and this is the actual config change.
Jira: (Skip unless you are MA staff)
https://jira.mass.gov/browse/DP-20161
To Test:
Testing this is going to be kind of a bear. You cannot test this locally--you get warnings about CSRF attacks. If you want to see it working:
[node:nid]for any node IDHere's the iframe url (containing the drupal macro)
dashboards.digital.mass.gov/superset/dashboard/basic-pages/?preselect_filters=%7B%221823%22%3A%20%7B%22node_id%22%3A%20%5B%22[node:nid]%22%5D%7D%2C%20%221542%22%3A%20%7B%22__time_range%22%3A%20%22Last%20month%22%7D%7D
Peer Review Checklist