-
Notifications
You must be signed in to change notification settings - Fork 107
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
Introduce validator to raise ModuleNotFoundError at dashboard creation #97
Introduce validator to raise ModuleNotFoundError at dashboard creation #97
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! 👍 Just an idea on how I would rewrite the Error message.
The link from the PR Description is wrong, so can you just update it? 😄. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for fixing this! Just a couple of fussy comments/questions, but I'm sure we can get this merged soon 🙂
....d/20231005_145819_maximilian_schulz_86_absence_of_openpyxl_library_causes_console_errors.md
Outdated
Show resolved
Hide resolved
…sence_of_openpyxl_library_causes_console_errors.md Co-authored-by: Antony Milne <49395058+antonymilne@users.noreply.github.com> Signed-off-by: Maximilian Schulz <83698606+maxschulz-COL@users.noreply.github.com>
Co-authored-by: Antony Milne <49395058+antonymilne@users.noreply.github.com> Signed-off-by: Maximilian Schulz <83698606+maxschulz-COL@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool!
Tested it.
Approve from my side.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just one suggestion but looks good 👍
Description
Screenshot
Checklist
Enable feature XXX ([#1](https://github.com/mckinsey/vizro/pull/1))
(if applicable)Types of changes
Notice
I acknowledge and agree that, by checking this box and clicking "Submit Pull Request":