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

Add SP Siteusers addById() method #3073

Closed
wants to merge 1 commit into from

Conversation

RoelVB
Copy link
Contributor

@RoelVB RoelVB commented Jun 19, 2024

Category

  • Bug fix?
  • New feature?
  • New sample?
  • Documentation update?

What's in this Pull Request?

This PR adds the addUserById() method to be able to add siteusers using their ID.

@patrick-rodgers
Copy link
Member

Hey, what does this do? Doesn't the user id come from when you add the user to the site? Can you help us understand what this update is for?

@RoelVB
Copy link
Contributor Author

RoelVB commented Jul 1, 2024

We have removeById() and removeByLoginName(), but for some reason for adding we only had a method for using the loginname.
This PR adds the option be able to also add users by their ID.

@RoelVB
Copy link
Contributor Author

RoelVB commented Jul 1, 2024

I now understand why this PR is confusing. This same method is used by site-groups here:

public get users(): ISiteUsers {
return SiteUsers(this, "users");
}

In that context the method makes sense, but it doesn't for siteusers.

@juliemturner
Copy link
Collaborator

So I think we're just going to close this as I don't think it's needed.

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.

3 participants