Skip to content
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

fix: url for update endpoint #449

Merged
merged 3 commits into from
Nov 18, 2022
Merged

fix: url for update endpoint #449

merged 3 commits into from
Nov 18, 2022

Conversation

SavvyShah
Copy link
Contributor

@SavvyShah SavvyShah commented Nov 4, 2022

Change _update endpoint.

Closes #448

Copy link
Member

@siddharthlatest siddharthlatest left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm (y)

@dmoskovtsov
Copy link

dmoskovtsov commented Nov 7, 2022

I went to preview to test the fix. Update works if you update from the table but not working when you update from "Blue" update button & editing json directly, following error appears:

"no handler found for uri [/search-dev/undefined/vBgsf0Vwm86t8nAWMZpqy?refresh=wait_for] and method [PUT]"

Screen Shot 2022-11-07 at 20 09 22

@dmoskovtsov
Copy link

dmoskovtsov commented Nov 17, 2022

@SavvyShah FYI the preview is down now, but the last time I tried to edit the product from the JSON Document (see the previous message) it was not working.
cc @siddharthlatest

@siddharthlatest siddharthlatest merged commit 7eb4154 into dev Nov 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

BUG - Can't update a document, _type is empty/undefined
3 participants