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

Field trip fixes July 30 #424

Merged
merged 6 commits into from
Aug 31, 2021
Merged

Field trip fixes July 30 #424

merged 6 commits into from
Aug 31, 2021

Conversation

landonreed
Copy link
Member

@landonreed landonreed commented Jul 30, 2021

Copy link
Collaborator

@binh-dam-ibigroup binh-dam-ibigroup left a comment

Choose a reason for hiding this comment

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

Looks good with minor suggestions. Passing _tripIsPlanned() to the status prop seems ambiguous, maybe that needs to be clarified.

lib/components/admin/field-trip-status-icon.js Outdated Show resolved Hide resolved
lib/components/admin/trip-status.js Outdated Show resolved Hide resolved
lib/components/form/call-taker/date-time-options.js Outdated Show resolved Hide resolved
@landonreed
Copy link
Member Author

Thanks @binh-dam-ibigroup. Comments addressed in a99744f

@landonreed landonreed assigned evansiroky and unassigned landonreed Aug 4, 2021
@evansiroky evansiroky assigned landonreed and unassigned evansiroky Aug 4, 2021
@landonreed
Copy link
Member Author

@binh-dam-ibigroup I changed it to ok in e8722a5

@landonreed landonreed merged commit 88828d3 into dev Aug 31, 2021
@github-actions
Copy link
Contributor

github-actions bot commented Sep 8, 2021

🎉 This PR is included in version 3.4.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants