Skip to content
This repository has been archived by the owner on Mar 12, 2020. It is now read-only.

Revisiting Invitation Pages after accepting the invitation #154

Merged
merged 12 commits into from
Sep 1, 2015

Conversation

tarebyte
Copy link
Member

This closes #144 and closes #143.

When a student accepts an invitation to an assignment individual or group it will redirect them as expected.

But if they try to accept that same invitation again, it should automatically show them their assignment URL.

Looking for code feedback, and thoughts.

/cc @johndbritton

@tarebyte
Copy link
Member Author

This is what it currently looks like when a student accepts an individual assignment.

assignment_invitations

@johndbritton
Copy link
Contributor

The flow looks right to me. Planning to tackle the design issues separately?

@johndbritton
Copy link
Contributor

I think this resolves #143, but I don't know if it fully addresses #144. Since this is a new assignment, you might have the case where the group member previously accepted assignment A and formed teams, but has not accepted assignment B (which is using the same group of teams).

@tarebyte
Copy link
Member Author

May as well tackle the design here as well.

@tarebyte
Copy link
Member Author

@johndbritton how does this look for the individual assignment invitation page?

screen shot 2015-08-30 at 21 38 26

@johndbritton
Copy link
Contributor

@tarebyte I think you should include the full app chrome with "Classroom for GitHub," organization name, and assignment name heading sections and the info and actions inside that:

For the copy, how about this:

Accepting this assignment will give you access to the repository-name repository in the organization-name organization on GitHub.

@johndbritton
Copy link
Contributor

I don't think it's necessary to explain that it will be created if it hasn't already been and that they will be given access if they haven't been. Telling them the end result should be clear enough.

@tarebyte
Copy link
Member Author

@johndbritton better?

screen shot 2015-08-31 at 10 48 06

screen shot 2015-08-31 at 10 48 55

@johndbritton
Copy link
Contributor

❤️ Looks great.

tarebyte added a commit that referenced this pull request Sep 1, 2015
Revisiting Invitation Pages after accepting the invitation
@tarebyte tarebyte merged commit 05c1561 into master Sep 1, 2015
@tarebyte tarebyte deleted the revisiting-invitation-pages branch September 1, 2015 04:09
@tarebyte
Copy link
Member Author

tarebyte commented Sep 1, 2015

@johndbritton merging this for now, shout if you have any feedback!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Using an existing group of teams for a new group assignment Using a group invitation twice
2 participants