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

Allow to Set the GroupId we got from SharePoint in existing Context for Group #442

Closed
wants to merge 2 commits into from

Conversation

czullu
Copy link

@czullu czullu commented May 12, 2021

My question in #440 was if there is something against reusing the Context i have to work with Groups as it will be possible with this change.

@jansenbe jansenbe self-assigned this May 12, 2021
@jansenbe
Copy link
Contributor

Hi @czullu , I've fixed this internally as you as a developer should not have to manually populate the metadata on the model objects. See 8442e95 for the commit with the fixes and test case. If you still experience issues then please open an issue. Thanks for making us aware of this bug!

@jansenbe jansenbe closed this May 12, 2021
@czullu czullu deleted the Enhance_PnPContext_SetGroupId branch May 25, 2021 06:35
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