-
Notifications
You must be signed in to change notification settings - Fork 30
feat: CHARTS 3840 Release sweep #10
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
Conversation
…el dependency, added quickstart to all readme's
…link to iframe example in the unauth readme
tomhollander
left a comment
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.
Changes look good - are you going to do the tenant swticheroo in this PR or separately?
Doesn't matter to me! :) Happy to merge this now and do that tomorrow when the tenant is upgraded. |
tomhollander
left a comment
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, good to merge then. I'll let you know as soon as the tenant is upgraded.
scottsidwell
left a comment
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.
👍🏻
JIRA ticket link
📈 task: CHARTS-3840
Description
Rebased off of Scott's current PR, sorry for the double up shown!
npm startcommand