-
Notifications
You must be signed in to change notification settings - Fork 79
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
API Remove GraphQL #1477
API Remove GraphQL #1477
Conversation
cab8f2e
to
c5ed1bc
Compare
9e27562
to
2afd9a1
Compare
f418bce
to
08efa29
Compare
7dd90f7
to
9ac18a4
Compare
3f950c5
to
ed5efa5
Compare
ed5efa5
to
096335d
Compare
Few problems found while testing edge cases:
I stopped checking at that point - it seems likely that all of the endpoints suffer from similar problems. |
Replicated this on 5.2, so looks like it's an existing issue |
fec2587
to
a55c7d1
Compare
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 new json error handling is great!
Found one problem when testing this time around.
Cannot cancel deleting a file
- Upload a file.
- Select the file.
- Click the
...
button - Click delete
- Try to cancel the modal (it won't go away, and there's no errors in the console, no network requests, nothing)
The fact this wasn't caught by behat means we are missing a test scenario for this.
a55c7d1
to
a7666b3
Compare
Have fixed and added behat tests for cancelling and closing the modal |
a7666b3
to
49bb344
Compare
49bb344
to
c34ad4e
Compare
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, works as expected locally.
Issue silverstripe/silverstripe-admin#1799