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

Allow workshop admins to edit Fit workshop assignment. #20663

Merged
merged 1 commit into from Feb 16, 2018

Conversation

aoby
Copy link
Contributor

@aoby aoby commented Feb 15, 2018

Make workshop assignment control more generic and clean up that code. Remove unused
parameters, and add tests.

Now, workshop assignments (summer and fit where applicable) only appear at the top of the detail view. In read mode they are displayed with a link or "Unassigned". In edit mode, they display selects with the choices. Both teacher and facilitator applications have summer assignments. Only facilitator applications have FIT assignments, and only workshop admins can edit these.

Unassigned:
image

Edit:
image

Assigning:
image

Assigned (saved):
image

Facilitators

image

Admins double-edit:
image

Partners can only edit the summer workshop:
image

@aoby
Copy link
Contributor Author

aoby commented Feb 15, 2018

FiT Weekend assignments

assignment control more generic and clean up that code. Remove unused
parameters, and add tests.
Copy link

@clareconstantine clareconstantine left a comment

Choose a reason for hiding this comment

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

this looks good to me!

@aoby aoby merged commit 4db7ced into staging Feb 16, 2018
@aoby aoby deleted the fit-weekend-assignment branch February 16, 2018 19:37
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.

None yet

2 participants