-
Notifications
You must be signed in to change notification settings - Fork 5
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
Resolve invitation error when user dose not exists any more #8027
Resolve invitation error when user dose not exists any more #8027
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not the right way to fix this issues.
The problem should be fixed by just using the Actor.lookup() instead of the PloneUserActor
in the my-invitations view
.
9a820eb
to
47e99d8
Compare
Moved the logic to the |
1784aca
to
7db16e6
Compare
# Add the invitation to the storage | ||
self.storage._write_invitations[invitation_id] = entry | ||
|
||
# Resolve the inviter from the invitation |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This test does not actually check whether the problem still exists or not. Instead of manually fetching the inviter and create the actor from it, you should call the my-invitations view with the test user.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✅
7db16e6
to
3bb6cee
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍 - but changelog entry is missing.
3bb6cee
to
d24147f
Compare
This pull request addresses a recurring AttributeError encountered by users in Teamraum when handling invitations. The error was traced to an issue where a non-existent inviter was being referenced, leading to a failure in the invitation process.
For TI-1036
Before:
After:
Checklist
Everything has to be done/checked. Checked but not present means the author deemed it unnecessary.