Skip to content
This repository has been archived by the owner on Aug 7, 2024. It is now read-only.

[BUG] <adds multiple milestones> #9687

Closed
1 task done
malay44 opened this issue Oct 30, 2023 · 7 comments · Fixed by #9713
Closed
1 task done

[BUG] <adds multiple milestones> #9687

malay44 opened this issue Oct 30, 2023 · 7 comments · Fixed by #9713
Assignees
Labels
🛠 goal: fix undefined Hacktoberfest Good issues for Hacktoberfest participants 🔢 points: 2 undefined 🏁 status: ready for dev You can asked for this issue to be assigned (if not already assigned)

Comments

@malay44
Copy link
Contributor

malay44 commented Oct 30, 2023

Has this bug been raised before?

  • I have checked "open" AND "closed" issues and this is not a duplicate

Where did you find this bug?

while adding milestone sectio

Version of BioDrop (for example "v1.2.3")

2.83.14

Description

when clicked multiple times on save button in milestone section, it sends many requests which results in adding of the same milestone multiple times

Screenshots

Screenshot 2023-10-30 at 9 13 16 PM

Do you want to work on this issue?

Yes

If you want to work on this issue...

By disabling the button when the request is sent.

@malay44 malay44 added 🚦 status: awaiting triage Waiting for maintainers to verify (please do not start work on this yet) 🛠 goal: fix undefined labels Oct 30, 2023
@github-actions
Copy link
Contributor

To reduce notifications, issues are locked until they are 🏁 status: ready for dev You can asked for this issue to be assigned (if not already assigned) and to be assigned. You can learn more in our contributing guide https://github.com/EddieHubCommunity/BioDrop/blob/main/CONTRIBUTING.md

@github-actions github-actions bot locked and limited conversation to collaborators Oct 30, 2023
@sital002
Copy link
Member

@malay44 Good spot assigning the issue to you.

@sital002 sital002 added 🏁 status: ready for dev You can asked for this issue to be assigned (if not already assigned) 🔢 points: 2 undefined Hacktoberfest Good issues for Hacktoberfest participants and removed 🚦 status: awaiting triage Waiting for maintainers to verify (please do not start work on this yet) labels Oct 30, 2023
@github-actions
Copy link
Contributor

The issue has been unlocked and is now ready for dev. If you would like to work on this issue, you can comment to have it assigned to you. You can learn more in our contributing guide https://github.com/EddieHubCommunity/BioDrop/blob/main/CONTRIBUTING.md

@github-actions github-actions bot unlocked this conversation Oct 30, 2023
@eddiejaoude
Copy link
Member

I was unable to replicate this bug. Milestones redirects after the save, similar to the links page. Are you trying to purposely break it by clicking multiple times before the request comes back successful and the page redirects?

If the above steps to reproduce are correct, it sounds like an edge case and would effect all manage pages and those would need to be updated accordingly too so they are consistent.

@sital002
Copy link
Member

sital002 commented Oct 31, 2023

I was unable to replicate this bug. Milestones redirects after the save, similar to the links page. Are you trying to purposely break it by clicking multiple times before the response comes back successful and the page redirects?

If the above steps to reproduce are correct, it sounds like an edge case and would effect all manage pages and those would need to be updated accordingly too so they are consistent.

If you spam the save button it would create the milestone multiple times what they are suggesting is to disable the button until the response comes so that there is only one request.

@malay44
Copy link
Contributor Author

malay44 commented Oct 31, 2023

I was unable to replicate this bug. Milestones redirects after the save, similar to the links page. Are you trying to purposely break it by clicking multiple times before the request comes back successful and the page redirects?

If the above steps to reproduce are correct, it sounds like an edge case and would effect all manage pages and those would need to be updated accordingly too so they are consistent.

I would also update other manage pages 😊.

@eddiejaoude
Copy link
Member

eddiejaoude commented Oct 31, 2023

I would also update other manage pages 😊.

Thank you 👍 I suggest you do 1 page first and create a draft PR and it can be reviewed, then you can apply it to all other pages afterwards if approved

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🛠 goal: fix undefined Hacktoberfest Good issues for Hacktoberfest participants 🔢 points: 2 undefined 🏁 status: ready for dev You can asked for this issue to be assigned (if not already assigned)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants