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 7 commits into
base: master
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.

@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 master May 17, 2024 15:56
@benjamin-antupit benjamin-antupit changed the base branch from dev to master January 1, 2025 08:58
@benjamin-antupit benjamin-antupit force-pushed the 399-reduce-redundancy-in-row-based-forms branch from c6412f7 to 1f4fdb0 Compare January 1, 2025 08:58
@coveralls
Copy link

Coverage Status

coverage: 90.757% (+0.005%) from 90.752%
when pulling 1f4fdb0 on 399-reduce-redundancy-in-row-based-forms
into 800a900 on master.

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