-
Notifications
You must be signed in to change notification settings - Fork 76
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
Refactor manageOverview.php
API endpoint
#1881
Refactor manageOverview.php
API endpoint
#1881
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.
LGTM. One small issue with saving a layout with no groups at all, but I don't imagine that's going to happen too often.
I've created an issue (#1889) for it and we can decide if it should be made a part of this change or a future update.
I'll make the changes in this PR. |
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.
Much better. Thanks for the instant fix!
This PR continues our ongoing effort to migrate our legacy API to Laravel controllers, and to convert our protractor tests to Cypress. Supersedes #1666. I also made Cypress query the value of APP_URL each time a test is run in case the value changes from one test to another. This PR also fixes #1889.