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

Clarify help text on trip assignment creation #4968

Merged
merged 1 commit into from
Dec 14, 2024

Conversation

kberzinch
Copy link
Member

@kberzinch kberzinch commented Dec 14, 2024

@@ -0,0 +1 @@
Only members that have recently paid dues are shown here by default. If you need to create an assignment for a member that hasn't paid dues, go to their user details page and click <strong>Create Trip Assignment</strong> there.
Copy link
Member

Choose a reason for hiding this comment

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

maybe remove "by default" for brevity?

also is "recently paid dues" simplifying some complicated logic? is it anyone considered active (vs. access active but not active? I just feel like the wording could be more precise

Copy link
Member Author

Choose a reason for hiding this comment

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

It is just access active, yes, but I don't think that's helpful information for non-technical users.

@@ -1 +1 @@
Only upcoming trips are shown here by default. If you need to create an assignment for a past trip, go to the trip details page and click <strong>Create Trip Assignment</strong> there.
Only upcoming trips in draft status are shown here by default. If you need to create an assignment for a past trip, go to the trip details page and click <strong>Create Trip Assignment</strong> there.
Copy link
Member

Choose a reason for hiding this comment

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

same here re "by default" unless I'm missing something

@kberzinch kberzinch merged commit b5b2c55 into main Dec 14, 2024
9 checks passed
@kberzinch kberzinch deleted the kristaps/travel-help-text branch December 14, 2024 23:16
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.

2 participants