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

feat(PeoplePicker): add new allowUnvalidated option to allow adding non-tenant users #1232

Merged
merged 2 commits into from
Jun 21, 2022

Conversation

JakeStanger
Copy link
Contributor

@JakeStanger JakeStanger commented Jun 13, 2022

Guest users can be identified by the userUnvalidated flag.

This is useful for then handling inviting users as guests.

Q A
Bug fix? [ ]
New feature? [x]
New sample? [ ]
Related issues?

What's in this Pull Request?

Adds new prop to the PeoplePicker component called allowUnvalidated. Setting this to true no longer filters out users which are "unvalidated", which in general means they do not exist in the tenant. This allows the picker to be used for example for an interface to invite guest users.

If a user is unvalidated, the isUnvalidated key will be true on the returned object.

Guest users can be identified by the `userUnvalidated` flag.

This is useful for then handling inviting users as guests.
@JakeStanger JakeStanger changed the title feat: add new allowUnvalidated option to allow adding non-tenant users feat(PeoplePicker): add new allowUnvalidated option to allow adding non-tenant users Jun 16, 2022
Updated comment
Copy link
Contributor Author

@JakeStanger JakeStanger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good spot, thanks :)

@joelfmrodrigues joelfmrodrigues merged commit 06f8ac5 into pnp:dev Jun 21, 2022
@JakeStanger JakeStanger deleted the feat/peoplepicker-guests branch June 21, 2022 22:45
@joelfmrodrigues
Copy link
Collaborator

Hi @JakeStanger , many thanks for the contribution. The PR has now been merged and changes will be available as part of the next release. You can also test the changes as part of the beta release that will be published automatically in a few minutes.

@joelfmrodrigues joelfmrodrigues added this to the 3.9.0 milestone Jun 21, 2022
@joelfmrodrigues
Copy link
Collaborator

Also, as a side note, I have updated the control docs to include the new property 🙂
Thanks again

@JakeStanger
Copy link
Contributor Author

Ah I completely forgot about docs, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants