-
Notifications
You must be signed in to change notification settings - Fork 26
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
[Canned Reports] Create visualisation on question #835
Comments
Please assign it to me @mahalakshme ! |
Hey @mahalakshme
p.s. code is almost ready for this! |
@ombhardwajj the above is fine, just hide the observations column, I think when compared to creating model in metabase, you can just create a view in the server codebase to use for this if needed. cc: @petmongrels based on yesterdays' discussion thought we can start back the work. |
Commit added @mahalakshme ! |
Yayy!! |
|
@petmongrels thought if creating dashboard and filters for it are not too complex, we can do them - to assess that - added creating one dashboard and filter to this card: #837 not the current(#835) card. My intention was to try to get things work end-end instead of going deep on one thing. Let me know if you think otherwise. |
ok. will review the code when done |
Code refactoring done , please review @petmongrels ! |
Testing related
|
…web packages or unnecessary dependencies.
AC:
Technical suggestions:
Metabase api link
Approximate payload that might work - Not tested:
Thoughts:
Old: Ignore
Inputs-Ignore-for analysis purpose and later reference:
The text was updated successfully, but these errors were encountered: