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

feat(Annotator): Avoid button spam [ADEV-38] #198

Merged
merged 119 commits into from
Jun 24, 2024

Conversation

ElodieENSTA
Copy link
Member

@ElodieENSTA ElodieENSTA commented Mar 22, 2024

To merge after #191

You were able to add 17x the same user in the campaign edition, only because the submit button remained available after click.

Now every button that submit a request are disabled while the request is done.

To test you can try spamming the buttons while using "Slow 3G" preset thanks to the network dev tools of Chrome

rvovard and others added 30 commits February 1, 2024 11:36
…ecated create-react-app (create / move / delete files)
Redirect login already logged
@ElodieENSTA ElodieENSTA changed the title feature(Annotator): Avoid button spam [ADEV-38] feat(Annotator): Avoid button spam [ADEV-38] Apr 30, 2024
@ElodieENSTA ElodieENSTA force-pushed the feature/ADEV-38-Anti-spam-sur-les-boutons branch from e8cac25 to 785399d Compare May 2, 2024 08:17
@ElodieENSTA ElodieENSTA force-pushed the feature/ADEV-38-Anti-spam-sur-les-boutons branch from 785399d to 43fdf60 Compare May 13, 2024 07:25
…i-spam-sur-les-boutons

# Conflicts:
#	backend/api/admin/__init__.py
#	backend/api/serializers/annotation/campaign/create.py
#	backend/api/serializers/annotation_task.py
#	backend/tests/views/annotation_campaign.py
#	frontend/cypress/e2e/0-APLOSE/1 - login.cy.js
#	frontend/package-lock.json
#	frontend/package.json
#	frontend/src/App.tsx
#	frontend/src/components/form/inputs/select.tsx
#	frontend/src/components/form/searchbar/searchbar.component.tsx
#	frontend/src/css/ionic-override.css
#	frontend/src/fonts/Exo_2/OFL.txt
#	frontend/src/slices/annotator/annotations.ts
#	frontend/src/view/annotation-campaign-detail.page.tsx
#	frontend/src/view/annotation-campaign-list.page.tsx
#	frontend/src/view/audio-annotator/audio-annotator.page.tsx
#	frontend/src/view/audio-annotator/components/bloc/confidence-indicator-bloc.component.tsx
#	frontend/src/view/audio-annotator/components/navigation-buttons.component.tsx
#	frontend/src/view/audio-annotator/components/spectro-render.component.tsx
#	frontend/src/view/create-campaign/blocs/annotations.bloc.tsx
#	frontend/src/view/create-campaign/blocs/import-modal/detectors-config-content.tsx
#	frontend/src/view/create-campaign/create-campaign.page.tsx
#	frontend/src/view/create-campaign/edit-campaign.page.tsx
#	frontend/src/view/dataset-list/dataset-list.page.tsx
#	frontend/src/view/dataset-list/modal-new-dataset.component.tsx
#	frontend/src/view/login.page.tsx
#	frontend/vite.config.ts
@ElodieENSTA ElodieENSTA merged commit 7c43273 into master Jun 24, 2024
3 checks passed
@ElodieENSTA ElodieENSTA deleted the feature/ADEV-38-Anti-spam-sur-les-boutons branch June 24, 2024 09:10
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.

2 participants