Skip to content

Commit

Permalink
Merge pull request #3542 from kasperbolarsen/develop
Browse files Browse the repository at this point in the history
Update bug report and index.md templates + new Q&A page
  • Loading branch information
wobba authored Feb 19, 2024
2 parents 071acaa + 1bcb87e commit ac0e348
Show file tree
Hide file tree
Showing 4 changed files with 60 additions and 1 deletion.
7 changes: 7 additions & 0 deletions .github/ISSUE_TEMPLATE/bug_report.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,8 @@ assignees: ''

---



### Check existing issues before logging new ones
If you are not using the latest version, triage your issue with the latest version before logging as we won't provide patches to older versions.

Expand All @@ -16,6 +18,11 @@ If you are not using the latest version, triage your issue with the latest versi

__Remove the above section before submitting, and removal of the below template will result in the issue being closed.__

### Follow the template below or we will have to delete this request!!
We can assist a lot faster if you provide the following information:



**Version used**
Ex: 4.8

Expand Down
45 changes: 45 additions & 0 deletions docs/Q&A.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,45 @@


**Q: In version 4.10.1 the LPC (Live Person Card) hover option became available as an option for the People Layout. What is the difference between LCP and Persona card? **

![LCP option](/assets/LCP.png "LCP option in the People Layout")





**A**: The Live Person Card does not require any additional Graph Permissions. The LPC can be customized to show additional fields from Entra ID, but not the same way as the pnp-people or mgt-person. However it will always show equal to any other people card shown in Microsoft 365.See https://learn.microsoft.com/en-us/graph/add-properties-profilecard.

----------------------

**Q: Is the deprecation of SharePoint-Add-in's affecting PnP Modern Search?**

**A**: No, as the project is built using the SharePoint Framework, not the deprecated add-in model.

-----------------------

**Q: Is the PnP Modern Search package certified by a 3rd party in order to ensure compliance with GDPR or similar requirements? **

**A**: No, it is up to you to review the source code in order to ensure compliance with any relevant requirements. The web parts do not store, process or log any data, thus GDPR is not directly relevant. Any privacy concern of data is up to how data is stored and protected at the source level, e.g. SharePoint.

-----------------------

**Q: Are the PnP Modern Search web parts logging data to a local or remote receiver? **

**A**: No, the PnP Modern Search web parts are not logging data to any receiver, not even telemetry data.

-----------------------

**Q: What is the ID or Name of the PnP Modern Search App Registration in Azure AD/Entra? **

**A**: There isn’t an App Registration nor Enterprise application as PnP Modern Search does not rely on an application entry. The solution uses FedAuth cookies from SharePoint when calling the SharePoint Search API, and uses the “SharePoint Online Client Extensibility” app registrations when calling Graph API’s.

The solution does elevate any permissions when calling the API’s as permissions are all of the type Delegated Permissions, meaning that the permissions are bound to the current user, not the PnP Modern Search solution.

-----------------------

**Q: I am concerned about what will happen if the project is abandoned. Will Microsoft take over the project? **

**A**: It is hard to predict the future, but a vast number of companies are using the PnP Modern Search web parts in their solutions. The solution is hosted on GitHub owned by the Microsoft Search team. Most likely these companies will either clone the project and make a commercial version or will provide the manpower needed to keep the project in maintenance mode.

-----------------------
Binary file added docs/assets/LCP.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
9 changes: 8 additions & 1 deletion docs/index.md
Original file line number Diff line number Diff line change
Expand Up @@ -86,9 +86,15 @@ Also, to help us to resolve your issue, you can include screenshots or error mes
- Errors displayed in the SharePoint console (pressing CTRL+F12)

## Issues, questions, feedback?

For any issue, question or feedback, please the [official GitHub repository](https://github.com/microsoft-search/pnp-modern-search/issues). We will be happy to help you!

## Q&A
We have a list of frequently asked questions available [here](q&a). If you have a question, it might be already answered there.






## About

Expand All @@ -110,3 +116,4 @@ Here is the list of main contributors of the PnP Modern Search (all versions inc
- Paolo Pialorsi (PiaSys.com) - [@PaoloPia](https://twitter.com/paolopia)
- Patrik Hellgren (SherparsGroupAB) - [@PatrikHellgren](https://twitter.com/patrikhellgren)
- Erfan Darroudi [@edarroudi](https://github.com/edarroudi)
- Kasper Larsen (Fellowmind) - [@kasperlarsen](https://twitter.com/kasperbolarsen)

0 comments on commit ac0e348

Please sign in to comment.