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.

Sign up for GitHub

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

Jump to bottom

Implements the access request configuration. Closes #1019 #1040

Merged
merged 1 commit into from
Nov 27, 2022
Merged

Implements the access request configuration. Closes #1019 #1040

merged 1 commit into from
Nov 27, 2022

Conversation

plamber
Copy link
Contributor

@plamber plamber commented Nov 24, 2022

Closes #1019

With SetAccessRequestAsync you can configure all three states:

  • Enabled with default settings
  • Specific mail
  • Not enabled

Since this is my first contribution, I might need feedback on how this feature could be further improved from an SDK standpoint.

Thank you,
Patrick

@plamber plamber mentioned this pull request Nov 24, 2022
Closed
@jansenbe jansenbe self-assigned this Nov 24, 2022
@jansenbe jansenbe added the area: model 📐 Related to the core SDK models label Nov 24, 2022
@jansenbe
Copy link
Contributor

Thanks a ton @plamber !

@jansenbe jansenbe merged commit e2f87df into pnp:dev Nov 27, 2022
jansenbe added a commit that referenced this pull request Nov 27, 2022
@jansenbe
Merge of #1040. Thanks @plamber
7f9de12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@jansenbe jansenbe

Labels
area: model 📐 Related to the core SDK models
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Support the manipulation of UseAccessRequestDefault and AccessRequestSiteDescription
2 participants
@plamber @jansenbe