Skip to content

Conversation

@lunika
Copy link
Member

@lunika lunika commented Oct 7, 2025

Purpose

A user can be invited and no control is made on the email case. Then,
when a new user is created, we are looking if there are pending
invitation and the lookup used is case sensitive. We change it using
__iexact which is case insensitive.

Proposal

  • 🐛(backend) filter invitation with case insensitive email
  • ♻️(backend) force saving invitation email in lowercase
@lunika lunika requested a review from AntoLC October 7, 2025 10:09
@lunika lunika self-assigned this Oct 7, 2025
@lunika lunika added bug Something isn't working backend labels Oct 7, 2025
@AntoLC AntoLC linked an issue Oct 7, 2025 that may be closed by this pull request
@lunika lunika force-pushed the fix/search_invitation_case_insensitive branch from 1e1ba8e to ba8eb3b Compare October 7, 2025 12:38
lunika and others added 3 commits October 7, 2025 14:51
A user can be invited and no control is made on the email case. Then, when a new user is created, we are looking if there are pending invitation and the lookup used is case sensitive. We change it using __iexact which is case insensitive.
We want to enforce that invitation email are saved in lower case.
When a user was searching for an email in the share modal, the search was case sensitive, so we were proposing to send an invitation to a new user when in fact the user was already registered. The search is now case insensitive, so the only choice is to add the existing user in the share list.
@lunika lunika force-pushed the fix/search_invitation_case_insensitive branch from ba8eb3b to 72f098c Compare October 7, 2025 12:51
@lunika lunika merged commit 72f098c into main Oct 7, 2025
20 of 22 checks passed
@lunika lunika deleted the fix/search_invitation_case_insensitive branch October 7, 2025 13:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend bug Something isn't working

3 participants