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

Adding is_manager feild #99

Merged
merged 12 commits into from
Jan 20, 2025
Prev Previous commit
Next Next commit
removing changes from other branch
Sahil590 committed Jan 15, 2025
commit 4970bdbcd3cb2254d7d11e99e950dc3b561f8996
1 change: 1 addition & 0 deletions imperial_coldfront_plugin/views.py
Original file line number Diff line number Diff line change
@@ -155,6 +155,7 @@ def accept_group_invite(request: HttpRequest, token: str) -> HttpResponse:
# Update group membership in the database.
group = ResearchGroup.objects.get(owner__pk=invite["inviter_pk"])
GroupMembership.objects.get_or_create(group=group, member=request.user)

return render(
request=request,
context={"inviter": group.owner, "group": group.name},
2 changes: 2 additions & 0 deletions tests/test_views.py
Original file line number Diff line number Diff line change
@@ -170,6 +170,8 @@ def test_get_valid_token(self, user_client, pi_group, user):
token = self._get_token(user.email, pi_group.owner.pk)
response = user_client.get(self._get_url(token))
assert response.status_code == HTTPStatus.OK
assert b"You have accepted a group invitation" in response.content
GroupMembership.objects.get(group=pi_group, member=user)

def test_get_valid_token_already_member(self, user_client, pi_group, user):
"""Test that the view doesn't duplicate group memberships."""