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.

Sign up for GitHub

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

Jump to bottom

feat: Metabase dashboard analytics module #4148

Closed
wants to merge 52 commits into from
Closed

feat: Metabase dashboard analytics module #4148

wants to merge 52 commits into from

Conversation

zz-hh-aa
Copy link
Collaborator

What does this PR do?

  • Creates services getDashboard, copyDashboard, updateFilter and generatePublicLink
  • Adds necessary types and tests
  • Creates new route for analytics module
  • Exports the Metabase client as a singleton instance (this also happens in 8f12f20 in feat: Metabase module controller and service #4072 but there were other changes in the same commit so I've isolated the change here)
  • Updates Swagger yaml

Why?

Following on from #4072, this should complete all the services we need to automate Metabase dashboard creation

This PR became a bit bigger than I'd anticipated... let me know if it'd be better to break it up!

ianjon3s and others added 30 commits December 18, 2024 17:54
@ianjon3s
fix: Feedback table alignment (#4089)
79d6b49
@jessicamcinchak @augustlindemer
chore: showInternalNotes in editor modals for portals (#4091)
6f0a2c7 
Co-authored-by: augustlindemer <118665588+augustlindemer@users.noreply.github.com>
@jamdelion
chore: setup power automate tokens for Gateshead (#4094)
e41f810
@ianjon3s
fix: Spacing of review tables (#4093)
4b7b450
@jessicamcinchak
feat: add audit table for data migration scripts (#4095)
14dfb4a
@RODO94
fix: add metabase_id to database scripts (#4100)
2632b72
@ianjon3s
fix: Responsive type sizes (#4099)
40a036c
@RODO94
feat: E2E Testing for Planning Constraints (#4063)
7df5452
@augustlindemer
chore: fix confirmation and invite to pay page default copy (#4101)
b7d7465
@freemvmt
[infrastructure] add explicit autoscaling logic to Hasura service (#4096
3421ae5 
)
@jessicamcinchak
fix: add logs column to temp_data_migrations_audit table and crea…
91b16ea 
…te relationship from `flow` to `lowcal_sessions` (#4102)
@freemvmt
[hasura] use wget instead of curl for hasuraProxy container health ch…
0d5e368 
…eck (#4103)
@jessicamcinchak @DafyddLlyr
feat: autocomplete data fields in the editor (#4062)
4392053 
Co-authored-by: Dafydd Llŷr Pearson <DafyddLlyr@gmail.com>
@DafyddLlyr
chore: Upgrade to Node 22 (LTS) (#4107)
6db8501
@DafyddLlyr
chore: Import planx-core as ESM (#4105)
8c864f7
@jessicamcinchak
chore: refactor public component wrappers for auto-answering into pre…
2984d2a 
…view `Node` (#4108)
@jessicamcinchak
fix: use ESM-compatible __dirname reference in e2e/tests/api-driven (…
9ab99a3 
…#4111)
@jessicamcinchak
migrate: find+replace Calculate output prop with fn (#4083)
bfec874
@jessicamcinchak
migrate: find+replace Option/Answer flag prop to flags and ensure…
6ea5801 
… always `string[]` type (#4098)
@jessicamcinchak @DafyddLlyr
migrate: bump planx-core (#4114)
7825c70 
Co-authored-by: Dafydd Llŷr Pearson <DafyddLlyr@gmail.com>
@jessicamcinchak
migrate: find+replace DrawBoundary dataFieldBoundary prop to fn a…
af2b1bb 
…nd remove `dataFieldArea` prop (#4085)
@DafyddLlyr
chore: Bump planx-core (#4117)
d794668
@jessicamcinchak
fix: ensure area is correctly displayed when navigating back to DrawB…
5088b53 
…oundary (#4119)
@jessicamcinchak
chore: bump planx-core (#4121)
f52c06f
@RODO94
chore: bump planx-core in Editor (#4124)
7eab061
@RODO94
fix: add guard clause for flow[nodeId] in autoAnswerableOptions (#4123)
eba45ca
@ianjon3s
refactor: Universal toggle button (#4115)
77906d0
@RODO94
feat: add has_send_component to flows table (#4113)
ff5d2b6
@RODO94
integrations: Doncaster Power Automate / S3 (#4125)
10a746e
@ianjon3s
fix: Toggle button visual order (#4128)
c9412ae
DafyddLlyr and others added 22 commits January 9, 2025 22:37
@DafyddLlyr
feat: Read dbUrl from config file, not remote pulumi (#4129)
a71b156
@DafyddLlyr
fix: Pulumi output syntax (#4131)
5fe9432
@DafyddLlyr
fix: Add password and host as explicit pulumi secrets (#4132)
4efe2d2
@DafyddLlyr
fix: Type errors in pulumi [skip pizza] (#4133)
c84b0ad
@DafyddLlyr
chore: Point to original db (#4134)
60210d0
@DafyddLlyr
chore: Switch db passwords (#4136)
db16a4c
@jessicamcinchak
fix: explicitly use URLSearchParams when constructing send to email m…
5420338 
…agic links (#4138)
@RODO94
feat: add has_send_component to seed scripts (#4127)
7f99af3
@DafyddLlyr
test: Migrate e2e mock flow (#4140)
7b5e136
@DafyddLlyr
chore: Match MUI & Emotion version numbers to planx-core (#4141)
93c9c45
@DafyddLlyr
chore: Bump planx-core (#4142)
75f7a5d
@DafyddLlyr
fix: Add optional chaining to flag check (#4146)
a3198bb
@jamdelion
feat: exclusive 'or' checklist with grouped options (#4116)
6aa1556
@zz-hh-aa
feat: metabase dashboard service and controller
ccf9f1d
@zz-hh-aa
feat: new metabase dashboard route
810581d
@zz-hh-aa
feat: dashboard route in swagger docs
02921c2
@zz-hh-aa
fix: handle camelCase snake_case conversion
c1f1ce2
@zz-hh-aa
fix: access correct param for filter update
412de60
@zz-hh-aa
feat: update route and param handling
cb5bf8c
@zz-hh-aa
feat: string processing to rename dashboards
74e70e9
@zz-hh-aa
test: dashboard services tests
6f58495
@zz-hh-aa
fix: dashboard service following tests
0b1eaf9
@zz-hh-aa zz-hh-aa requested a review from a team January 14, 2025 17:20
@zz-hh-aa zz-hh-aa force-pushed the oz/dashboard-controller-service branch from 9db16ac to 0b1eaf9 Compare January 14, 2025 17:29
@zz-hh-aa zz-hh-aa changed the base branch from main to oz/analytics-metabase-collection January 14, 2025 17:29
@zz-hh-aa zz-hh-aa marked this pull request as draft January 14, 2025 17:30
@zz-hh-aa zz-hh-aa closed this Jan 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@zz-hh-aa @ianjon3s @jessicamcinchak @jamdelion @RODO94 @augustlindemer @freemvmt @DafyddLlyr