-
Notifications
You must be signed in to change notification settings - Fork 10
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
add in check in cta component #147
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
small nit but otherwise this is awesome!
@@ -7,6 +7,10 @@ def index | |||
@standup_meetings = @standup_meeting_group.standup_meetings | |||
.includes(:user) | |||
.where(meeting_date: @meeting_date) | |||
@current_user_standup_meeting = @standup_meetings.find do |meeting| |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
small nit, can you use detect
. Unsure if this is defined in our rubocop rules but find
/where
etc I associate with ActiveRecord while detect
and select
are arrays.
This PR adds in the check in cta component. This CTA is meant to be a reasuable that directs user to check in if they haven't, edit their response if they have submitted a startup meeting, and edit their response if they skipped their startup meeting.
One caveat here -- this PR does not handle the empty state wherein an there is no standup meeting defined for the date in question. To enable this functionality would require handling how the
StandupMeeting::SkipsController
andStandupMeeting::CompletionsController
deal with unpersistedStandupMeeting
s, as currently rendering this component depends onStandupMeeting::ActionButtonComponent
, which expects that the standup meeting passed to it is persisted.Screen shots: