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 the updated credential to an already existing entity in SharePoint integration #202

Merged
merged 1 commit into from
Jul 24, 2023

Conversation

leofmds
Copy link
Contributor

@leofmds leofmds commented Jul 24, 2023

If an existing SharePoint credential is updated by the processAuthorizactionCallback, the updated credential was not being saved. This PR addresses that.

@linear
Copy link

linear bot commented Jul 24, 2023

@leofmds leofmds requested review from roboli and seanspeaks July 24, 2023 19:57
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@ghost
Copy link

ghost commented Jul 24, 2023

👇 Click on the image for a new way to code review

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map legend

@leofmds leofmds added the bug Something isn't working label Jul 24, 2023
@leofmds leofmds merged commit a1da0de into main Jul 24, 2023
@seanspeaks
Copy link
Contributor

🚀 PR was released in @friggframework/api-module-microsoft-sharepoint@0.0.7 🚀

@seanspeaks seanspeaks added the released This issue/pull request has been released. label Jul 24, 2023
@seanspeaks seanspeaks deleted the feature/lef-545-fix-sharepoint-module-entity-update branch August 17, 2023 21:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working released This issue/pull request has been released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants