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

Asset dont be saved when the field is added after a first save. #476

Closed
stephanedemotte opened this issue Jun 10, 2022 · 6 comments
Closed

Comments

@stephanedemotte
Copy link

Description

Hello when i have asset field inside a super table, the asset field don't want to be save.

Steps to reproduce

  1. Create a super table with a text field (make it static field)
  2. Add the field inside a Category
  3. Create a category and fill the text field
  4. add an asset field inside the super table field
  5. try to add and asset inside the asset field and save
  6. the asset is not saved

Additional info

  • Plugin version: 3.0.0-beta.5
  • Craft version: 4.0.4
  • Multi-site: No
@engram-design
Copy link
Member

I can't replicate this sorry, and I've tried a few different layouts options just in case.

Our of interest, is this only when adding an asset field, or the same situation for other fields, when adding to a Super Table field that already has a value?

@engram-design
Copy link
Member

Hmmm, this actually seems an issue with saving field content in general, only after applying it from a provisional draft. I was testing by saving instantly, because waiting for the provisional draft to be saved (where you see the "Showing your unsaved changes" notice). But after making changes and waiting for that notice and applying save, no changes are saved - even for the text field.

But I'm also noticing the same behaviour with Matrix, so may need to investigate further.

@mmikkel
Copy link
Contributor

mmikkel commented Jun 11, 2022

Looks a lot like the issue described in craftcms/cms#11419.

@engram-design
Copy link
Member

Good pick @mmikkel we've made that same change on our end in the craft-4 branch.

@stephanedemotte
Copy link
Author

@engram-design new release soon ?

@engram-design
Copy link
Member

Fixed in 3.0.0-beta.6

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

No branches or pull requests

3 participants