-
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
FOUR-15487:Phase1: Remove Analytics chart from future and existing customers in dev (QuickSight related) #1585
Conversation
…stomers in dev (QuickSight related)
Passing run #690 ↗︎Details:
Review all test suite changes for PR #1585 ↗︎ |
QA server K8S was successfully deployed https://ci-9a8ef20acc.engk8s.processmaker.net |
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.
The following observations were found :
- The sizes of the cards are different (ANALYTICS CHART height is higher)
- The Welcome user is above the avatar
- The image in Anakytics chart is not responsive
https://github.com/ProcessMaker/screen-builder/assets/21198450/c6d3ad88-a185-46a9-b369-b184886dfd94
QA server K8S was successfully deployed https://ci-9a8ef20acc.engk8s.processmaker.net |
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.
The WELCOME was corrected, and the others observations works as expected behavior
@fagubla
}, | ||
methods: { | ||
openExternalLink() { | ||
window.open("/package-analytics-reporting", "_blank"); | ||
}, | ||
verifyChart() { | ||
this.showChart = this.graphic.name === "PM Analytics Chart" || this.graphic.name === "PM Analytics Dashboard"; |
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.
9a4f922
to
bfdfbd3
Compare
QA server K8S was successfully deployed https://ci-9a8ef20acc.engk8s.processmaker.net |
|
Issue & Reproduction Steps
Remove Analytics chart from future and existing customers in dev (QuickSight related)
Expected behavior:
Actual behavior:
Solution
How to Test
Test the steps above
Related Tickets & Packages
Code Review Checklist
ci:deploy
ci:next
ci:package-dynamic-ui:feature/FOUR-15487