new sample for MS Flow to call MSGraph with AppOnly permissions - REST & Webhooks

microsoftgraph / msgraph-community-samples

Repo for community contributed samples

MIT License

56 stars 22 forks source link

new sample for MS Flow to call MSGraph with AppOnly permissions - REST & Webhooks #13

Open johnnliu opened 6 years ago

johnnliu commented 6 years ago

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

What's in this Pull Request?

Please describe the changes in this PR. Sample description or details around bugs which are being fixed. This PR includes 3 ZIP files that can be imported into Microsoft Flow.

Flow Maker will need to provide an AppOnly ClientID, ClientSecret and TenantID to configure the Flows to periodically read from the tenant's Groups resource and synchronize it with a SharePoint List. Two additional Flows setup a Groups webhook on (recurring 3 day expiry) and listens to Groups changes to re-sync the SP List.

andrewconnell commented 6 years ago

Thanks for the submission... can you please take care of the following and update the PR? As is, there are a few issues with getting this listed for others to easily find it and easily understand what it is.

update the ./samples/README.md with a link pointing to this sample
update the sample's README.md to give it a more accurate title & ? The current title looks like a copy-paste error (Microsoft Graph Webhooks Sample for ASP.NET Core)

In the demo you showed in the MS Graph community call, seemed like this was more about Flow vs. an ASP.NET Core sample... or am I mistaken?

I'd also suggest providing more details in the sample's README.md for people to understand what this is and how they can implement it... see how the other samples' readme pages are setup.

russgove commented 6 years ago

should include a link to todays recoring when available

jthake commented 6 years ago

@johnnliu I would also assume that people don't know how to use these zips. can you provide links to tutorials on how to import these and steps needed to set up App IDs etc.

johnnliu commented 6 years ago

@andrewconnell have a look again - I've added references to recording, blog post and steps to import Flow zip package.

andrewconnell commented 6 years ago

FYI @johnnliu - @jthake has taken over reviewing submissions to this shared repo...

jthake commented 5 years ago

@johnnliu this totally dropped off my radar. But is this duplicative to the training lab we've just written with you guys?