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.

Sign up for GitHub

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

Jump to bottom

Possible Text List Duplication #9

Open
jcuenod opened this issue Feb 13, 2019 · 3 comments
Open

Possible Text List Duplication #9

jcuenod opened this issue Feb 13, 2019 · 3 comments

Comments

@jcuenod
Copy link
Member

jcuenod commented Feb 13, 2019

I just got four parallel texts on the client. I don't know what the response looked like but it seems worth de-duplicating that array in the request just in case.
@jcuenod jcuenod changed the title Text List Possible Text List Duplication Feb 13, 2019
@jcuenod
Copy link
Member Author

jcuenod commented Mar 22, 2020

It's a double click issue. If you click the button fast enough (before page refresh) the add button will process multiple times because it's not just toggling the text on and off.

@jcuenod
Copy link
Member Author

jcuenod commented Mar 22, 2020

Although when I removed a non-duplicated text, I got another copy of the duplicated one... strange

@jcuenod
Copy link
Member Author

jcuenod commented Mar 24, 2020

Yes, I'm pretty sure that jcuenod/parabible-client@6e645f5 fixes parabible/parabible-production-client#31.

Still, I'm going to leave open so that we de-dup the array on the server side.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jcuenod