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

Content with multiple arrays updates incorrectly #167

Closed
Paulsulli92 opened this issue Jun 23, 2017 · 3 comments
Closed

Content with multiple arrays updates incorrectly #167

Paulsulli92 opened this issue Jun 23, 2017 · 3 comments

Comments

@Paulsulli92
Copy link

Paulsulli92 commented Jun 23, 2017

I would like to have a content type with multiple arrays but it seem that it does not update properly. I made a content type with this command:
gen c listtest name:string list1:"[]string" list2:"[]string"
Then built and ran and in the web editor added two entries to each list. Upon saving, some of the entries in the first list changed to entries from the other list. I am also unable to remove list entries: upon hitting save, removed entries reappear. This was done with no changes to the listtest.go file.

@nilslice
Copy link
Contributor

This is related to #59 - unfortunately I haven't had time to dig in and fix. I believe it is a JavaScript bug though, from within the editor package. The Go code for the RepeatController has some JavaScript that re-names the name attributes of each repeated input.

If you get a chance to look into this, please let me know and feel free to ask any questions you have about the code. I'm on the road today and away through this weekend so I may be a bit slow to respond.

@nilslice
Copy link
Contributor

@Paulsulli92 there is an active contribution pending that fixes this issue. Give me a day or so and I'll ping you when its ready.

nilslice added a commit that referenced this issue Jun 26, 2017
fixed problem with saving multiple repeaters closes #59 closes #167
@nilslice
Copy link
Contributor

This should be good to go now -- if you find any more issues with it please let me know!

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

2 participants