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

399 reduce redundancy in row based forms #664

Open
wants to merge 8 commits into
base: dev
Choose a base branch
from

Conversation

benjamin-antupit
Copy link
Collaborator

In crew checkin/out, select single radio buttons by default and autofill when only one option is accepted. In crew hours and CC assignments for events with one category of services, the category is selected by default.

When there's a single option available in the crew check in/check out forms, it is automatically selected as default
Also, re-activate buttons for individual event check ins
@benjamin-antupit benjamin-antupit added this to the Summer 2022 milestone Jun 4, 2022
@benjamin-antupit benjamin-antupit requested a review from Muirrum June 4, 2022 05:55
@benjamin-antupit benjamin-antupit self-assigned this Jun 4, 2022
@benjamin-antupit benjamin-antupit linked an issue Jun 4, 2022 that may be closed by this pull request
2 tasks
@benjamin-antupit benjamin-antupit changed the base branch from master to dev September 18, 2022 18:47
@coveralls
Copy link

coveralls commented Sep 18, 2022

Coverage Status

coverage: 91.082% (+0.005%) from 91.077%
when pulling c6412f7 on 399-reduce-redundancy-in-row-based-forms
into 6384c05 on dev.

@@ -461,8 +461,9 @@ def hours_prefill_self(request, id):
if not event.ccinstances.exists():
raise PermissionDenied
if event.hours.filter(user=request.user).exists():
messages.add_message(request, messages.ERROR, 'You already have hours for this event.')
messages.add_message(request, messages.ERROR, 'You are already checked in to this event.')
Copy link
Member

Choose a reason for hiding this comment

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

can you test to make sure that this does the expected? It looks like it'll error whenever a member has hours

site_tmpl/admin.html Outdated Show resolved Hide resolved
@alextannenbaum alextannenbaum deleted the branch dev May 17, 2024 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: In progress
Development

Successfully merging this pull request may close these issues.

Reduce redundancy in row-based forms
4 participants