Skip to content

Reject participants on failed training, add rejection reason, fix redux state getters, add cleaned duration to csv export and table view#464

Merged
JackWilb merged 7 commits into
devfrom
small-optimzations
Sep 12, 2024
Merged

Reject participants on failed training, add rejection reason, fix redux state getters, add cleaned duration to csv export and table view#464
JackWilb merged 7 commits into
devfrom
small-optimzations

Conversation

@JackWilb

@JackWilb JackWilb commented Sep 9, 2024

Copy link
Copy Markdown
Contributor

Does this PR close any open issues?

Closes #461
Closes #450
Closes #216

Give a longer description of what this PR addresses and why it's needed

Added multiple features/fixes:

  • Reject participants on failed training
    • With rejection reason, "failed training"
  • add rejection reason
    • Identifies why people were rejected (optional)
    • In this refactor, added number to the button so you can see how many people will be rejected
  • fix redux state getters
    • Addressed a console error here
  • add cleaned duration to csv export and table view
    • Automatically handles the window events for visibility so that an analyst can see how long participants were looking at the tab.

Provide pictures/videos of the behavior before and after these changes (optional)

Screenshot 2024-09-09 at 3 41 44 PM

Are there any additional TODOs before this PR is ready to go?

No

@JackWilb JackWilb merged commit 9f41cae into dev Sep 12, 2024
@JackWilb JackWilb deleted the small-optimzations branch September 12, 2024 20:25
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.

1 participant