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

added multi-person-favorite sample #691

Merged
merged 3 commits into from
Aug 5, 2023
Merged

Conversation

AlexanderHenkel
Copy link
Contributor

Q A
Bug fix? no
New sample? yes
Related issues?

What's in this Pull Request?

This sample let a user add/remove a list item as a favorite, by adding the user name to a multi-person column.

@tecchan1107 tecchan1107 self-assigned this Aug 2, 2023
@tecchan1107 tecchan1107 merged commit a29b9bd into pnp:master Aug 5, 2023
@tecchan1107
Copy link
Collaborator

I have done the merge! Please check out the following link and let me know if you have any questions. (I made some minor adjustments when merging.)
https://github.com/pnp/List-Formatting/tree/master/column-samples/multi-person-favorite

Thank you so much for the awesome sample🤩🤩✨

@tecchan1107
Copy link
Collaborator

If you are interested, please register for the following program. When you register, you will get a digital badge!
M365 Platform & Power Platform Community Recognition Program

※Even if you register, it will not be distributed immediately. It will take some time before it is distributed.

@BusinessRefinery
Copy link

Hello, firstly thanks for putting this together.

I've been using this for a while now. We've just come across an issue where if a staff leaves the organisation (now unlicensed) who has favourited the list item (assume the same applies to any people column) then it errors to add any additional favourites for other users.

Error states - "We couldn't find an exact match". When you remove the former staff member it resolves the issue. So I'm going to have to write a script to bulk remove people from the column.

Have you come across this issue?

image

@tecchan1107
Copy link
Collaborator

Hi @BusinessRefinery .
I will try to verify if I can reproduce this on my end. Also, I will try to see if I can use PowerShell to bulk delete specified users.

@tecchan1107
Copy link
Collaborator

Hi @BusinessRefinery .

Sorry for the delay. I was able to reproduce the same phenomenon in my environment. I would like to inform you that I have reported this phenomenon to Microsoft.
SharePoint/sp-dev-docs#9670

@BusinessRefinery
Copy link

Thanks @tecchan1107

Glad it isn't just me. Hope MS resolve that one.

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

Successfully merging this pull request may close these issues.

3 participants