cli-microsoft365 icon indicating copy to clipboard operation
cli-microsoft365 copied to clipboard

New command: `booking business get`. Closes #3534

Open Jwaegebaert opened this issue 3 years ago • 5 comments

Closes #3534

Jwaegebaert avatar Jul 28 '22 14:07 Jwaegebaert

Fast as always @Jwaegebaert 👏 thanks a lot, we'll review asap.

martinlingstuyl avatar Jul 28 '22 16:07 martinlingstuyl

Hi @Jwaegebaert, looks great. Thank you. I would wait for #3532 to complete before merging this one. @nanddeepn do you have an expected ETA?

@waldekmastykarz: Bookings.Read.All is required to perform the activities. Could you please ensure that the permissions are assigned to the Azure AD multi-tenant application?

Thank you, Patrick

plamber avatar Aug 05 '22 03:08 plamber

Hi @plamber I will try to complete #3532 by Monday (8 August). Hope that works.

nanddeepn avatar Aug 05 '22 07:08 nanddeepn

Hi @plamber PR https://github.com/pnp/cli-microsoft365/pull/3569 is raised for https://github.com/pnp/cli-microsoft365/issues/3532

nanddeepn avatar Aug 08 '22 10:08 nanddeepn

Thank you @nanddeepn

Once @waldekmastykarz gave the permissions to the app I am going to merge the changes

plamber avatar Aug 08 '22 11:08 plamber

Bookings.Read.All is added to the PnP Management Shell AAD app reg

waldekmastykarz avatar Aug 18 '22 08:08 waldekmastykarz