Bail out with an error message if invitee is already in the phonebook

VERIFIED WORKSFORME

Status

Participation Infrastructure
Phonebook
P2
enhancement
VERIFIED WORKSFORME
7 years ago
6 years ago

People

(Reporter: Tobbi, Assigned: tofumatt)

Tracking

Details

(Whiteboard: [l10n impact])

Attachments

(1 attachment)

(Reporter)

Description

7 years ago
I just assumed this functionality would exist and wrote an automated testcase for it. Now, it's your turn to implement this so that my automated testcase does not fail (that would obviously make both of us, me and my testcase, very sad:

When inviting a user, we should check the database whether the email address of the invitee is already there. Then we don't want to proceed, but bail out with an error message.

Updated

7 years ago
Priority: -- → P2
Target Milestone: --- → 1.2
Whiteboard: [l10n impact]
(Assignee)

Updated

7 years ago
Assignee: nobody → tofumatt
(Assignee)

Comment 1

7 years ago
This is in the code... maybe this bug simply wasn't updated.

See the test case here:

https://github.com/mozilla/mozillians/blob/master/apps/phonebook/tests/test_invites.py#L71
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → WORKSFORME
Created attachment 577288 [details]
screenshot

QA verified on prod. An appropriate error message is presented to the user when trying to send an invite to an already vouched for user.

Updated

7 years ago
Status: RESOLVED → VERIFIED

Updated

6 years ago
Component: mozillians.org → Phonebook
Product: Websites → Community Tools
QA Contact: mozillians-org → phonebook
Target Milestone: 1.2 → ---
Version: unspecified → other
You need to log in before you can comment on or make changes to this bug.