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

Enable ability for users to select tasks they mapped #5953

Merged
merged 2 commits into from
Sep 5, 2023

Conversation

HelNershingThapa
Copy link
Contributor

@HelNershingThapa HelNershingThapa commented Jun 28, 2023

Resolves #5435

Changes Made

  • Added a new "Deselect and Validate" button to the task lock error modal, allowing users to deselect tasks they have mapped for validation.
  • Conditionally displayed action buttons based on the user's selected tasks and mapping status.
  • Abstracted the action buttons into a new LockErrorButtons component, which also handles task deselection and validation logic.
  • Updated existing test cases to account for the new button behavior.

Peek 2023-08-23 14-26

@sonarcloud
Copy link

sonarcloud bot commented Jun 28, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Copy link
Contributor

@Aadesh-Baral Aadesh-Baral left a comment

Choose a reason for hiding this comment

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

LGTM

@ramyaragupathy ramyaragupathy self-requested a review August 15, 2023 09:17
@HelNershingThapa HelNershingThapa marked this pull request as draft August 16, 2023 10:00
@ramyaragupathy
Copy link
Member

Per chat w/ @HelNershingThapa and @Aadesh-Baral following are the next steps:

  • one task selected/multiple tasks selected and if it has self-mapped tasks -> then on clicking validate button show a pop-up prompting user.
  • Also indicate the task numbers in the pop-up? and then give an option to deselect the self mapped tasks and continue validation for the rest of the tasks. In case of a single task that is self-mapped, let users close the pop-up with no option to deselect.

- Emend popup text to "This task was mapped by you"
- Remove the not-allowed cursor implementation
- Enhances the `LockError` modal by introducing a "Deselect and Continue" button. The button is displayed when a `CannotValidateMappedTask` task lock error occurs. The "Close" button is shown with the primary variant when only a single task is selected.
- Conditionally displayed the buttons based on the user's selected tasks and mapping status.
- Separated the action buttons into a new component named `LockErrorButtons`.
- Moved the logic for handling task deselection and validation into the `LockErrorButtons` component.
- Updated the existing test cases to reflect the new changes made in the `LockErrorButtons` component.
@HelNershingThapa HelNershingThapa force-pushed the enhancement/5435-click-mapped-task branch from 9d42fc7 to f1f1acd Compare August 23, 2023 08:47
@sonarcloud
Copy link

sonarcloud bot commented Aug 23, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@HelNershingThapa HelNershingThapa marked this pull request as ready for review August 23, 2023 08:56
@ramyaragupathy
Copy link
Member

For now, we are not including task numbers in the pop-up and keep the message minimal. This can be later added if the users find it beneficial.

Copy link
Collaborator

@emi420 emi420 left a comment

Choose a reason for hiding this comment

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

I have tested this feature and reviewed the code, all good!

@HelNershingThapa HelNershingThapa merged commit df1dd6d into develop Sep 5, 2023
9 checks passed
@HelNershingThapa HelNershingThapa deleted the enhancement/5435-click-mapped-task branch September 5, 2023 05:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support ability to view info about task you mapped by clicking on the map
4 participants