[PR #3137] [MERGED] fix: fixed issue with team-invitations and new user accounts #4241

Closed
opened 2026-03-17 01:47:49 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/hoppscotch/hoppscotch/pull/3137
Author: @balub
Created: 6/14/2023
Status: Merged
Merged: 6/15/2023
Merged by: @AndrewBastin

Base: release/2023.4.7Head: fix/team-invitation-bug


📝 Commits (1)

  • 363cd29 fix: fixed issue with team-invitations and new user accounts

📊 Changes

2 files changed (+2 additions, -2 deletions)

View changed files

📝 packages/hoppscotch-backend/src/team-invitation/team-invite-viewer.guard.ts (+1 -1)
📝 packages/hoppscotch-backend/src/team-invitation/team-invitee.guard.ts (+1 -1)

📄 Description

Closes issue #3090

Description

This PR fixes a bug with the auth guards present in the TeamInvitation module where new users trying to join a team with a valid invitation could not do so and be presented with the BUG_AUTH_NO_USER_CTX error instead.

How to reproduce bug for testing

Have a normal running instance of SH running with one valid user account then:

  • Create a new team and Invite a new user whose account does not exist in the DB into the team.
  • Create the account for the user to whom you sent the invite link in the earlier step.
  • Make sure to be logged in as the newly created user click on the button in the email you received before to join the team.
  • You should see the error being thrown.

Issue Number

HBE-227

Checks

  • My pull request adheres to the code style of this project
  • My code requires changes to the documentation
  • I have updated the documentation as required
  • All the tests have passed

Additional Information

Nil


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/hoppscotch/hoppscotch/pull/3137 **Author:** [@balub](https://github.com/balub) **Created:** 6/14/2023 **Status:** ✅ Merged **Merged:** 6/15/2023 **Merged by:** [@AndrewBastin](https://github.com/AndrewBastin) **Base:** `release/2023.4.7` ← **Head:** `fix/team-invitation-bug` --- ### 📝 Commits (1) - [`363cd29`](https://github.com/hoppscotch/hoppscotch/commit/363cd291f798d41306bda20593b2efbbbb0db2ed) fix: fixed issue with team-invitations and new user accounts ### 📊 Changes **2 files changed** (+2 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `packages/hoppscotch-backend/src/team-invitation/team-invite-viewer.guard.ts` (+1 -1) 📝 `packages/hoppscotch-backend/src/team-invitation/team-invitee.guard.ts` (+1 -1) </details> ### 📄 Description <!-- Thanks for creating this pull request 🤗 Please make sure that the pull request is limited to one type (docs, feature, etc.) and keep it as small as possible. You can open multiple prs instead of opening a huge one. --> <!-- If this pull request closes an issue, please mention the issue number below --> <!-- Issue # here --> Closes issue #3090 ### Description <!-- Add a brief description of the pull request --> This PR fixes a bug with the auth guards present in the TeamInvitation module where new users trying to join a team with a valid invitation could not do so and be presented with the *BUG_AUTH_NO_USER_CTX* error instead. ### How to reproduce bug for testing Have a normal running instance of SH running with one valid user account then: - Create a new team and Invite a new user whose account does not exist in the DB into the team. - Create the account for the user to whom you sent the invite link in the earlier step. - Make sure to be logged in as the newly created user click on the button in the email you received before to join the team. - You should see the error being thrown. ### Issue Number HBE-227 ### Checks <!-- Make sure your pull request passes the CI checks and do check the following fields as needed - --> - [x] My pull request adheres to the code style of this project - [ ] My code requires changes to the documentation - [ ] I have updated the documentation as required - [x] All the tests have passed ### Additional Information <!-- Any additional information like breaking changes, dependencies added, screenshots, comparisons between new and old behaviour, etc. --> Nil --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-17 01:47:49 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/hoppscotch#4241
No description provided.