Skip to content

Conversation

@jettapat-metier
Copy link
Collaborator

Why did you create this PR

What did you do

  • Integrate upload storage in richtext editor

Screenshots / Recordings

  • Uploading Image RichText Feature
uploading-image-richtext.mov
  • Uploading Feature still Wroked
uploading-still-worked.mov

Checklist

  • Self-reviewed your code
  • Wrote coverage tests
  • Added screenshots or recordings if applicable

@changeset-bot
Copy link

changeset-bot bot commented Sep 16, 2025

🦋 Changeset detected

Latest commit: 252a859

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 8 packages
Name Type
@genseki/react Patch
@example/erp Patch
@example/ui-playground Patch
@genseki/next Patch
@genseki/plugins Patch
@genseki/prisma-generator Patch
@genseki/react-query Patch
@genseki/rest Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

context,
{
method: 'DELETE',
method: 'GET',
Copy link
Member

Choose a reason for hiding this comment

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

Why does delete endpoint has method GET na?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This endpoint just for generate signed url object to process. It's should be GET instead of DELETE

@saenyakorn saenyakorn merged commit 9bf60f8 into main Sep 19, 2025
1 check passed
@saenyakorn saenyakorn deleted the feat/update-image-rich-text branch September 19, 2025 07:36
This was referenced Sep 19, 2025
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.

5 participants