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

Added two_fields_in_one_model demo. #62

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

dzhuang
Copy link
Owner

@dzhuang dzhuang commented Apr 22, 2024

@phoebebright I tried to used your code to create another demo app which address your concern. However I'm now stuck at make it work since I didn't find the view code which render the gallery in the code you provided at #59 . Can you add more code here? FYI, the new demo name is demo_multiple_fields_1_model.

@dzhuang dzhuang force-pushed the two_gallery_fields_in_one_model branch from 7b55773 to fe76ca6 Compare April 22, 2024 04:09
Copy link
Owner Author

@dzhuang dzhuang left a comment

Choose a reason for hiding this comment

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

There should be forms and views for the MyGallery model

@sphrlbd
Copy link

sphrlbd commented Apr 22, 2024

Sure
galeryfiled test.zip

I didn't write the views, only used it on admin panel, and after I upload one image it will be shown on both field and uploaded to MyImage2

@sphrlbd
Copy link

sphrlbd commented Apr 22, 2024

But i see a new problem
image

@sphrlbd
Copy link

sphrlbd commented Apr 22, 2024

If you want, we can contact on Discord or something else

@dzhuang
Copy link
Owner Author

dzhuang commented Apr 22, 2024

@sphrlbd I've updated the code, and I can now understand the issue you proposed.

@dzhuang
Copy link
Owner Author

dzhuang commented Apr 22, 2024

Hi, @sphrlbd . I think I've fixed the bug (in new version v2.2.0). Can you switch to this Branch to see if it works as expected (in admin)? Thanks.

@sphrlbd
Copy link

sphrlbd commented Apr 22, 2024

Yeah, I'll test it. Thanks

@sphrlbd
Copy link

sphrlbd commented Apr 23, 2024

It fixed thanks

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.

None yet

2 participants