Skip to content

Updates spo tenant appcatalog add reference to deprecated spo site classic add command. Closes #3457#3459

Closed
martinlingstuyl wants to merge 1 commit intopnp:mainfrom
martinlingstuyl:updateDeprecatedReference
Closed

Updates spo tenant appcatalog add reference to deprecated spo site classic add command. Closes #3457#3459
martinlingstuyl wants to merge 1 commit intopnp:mainfrom
martinlingstuyl:updateDeprecatedReference

Conversation

@martinlingstuyl
Copy link
Copy Markdown
Contributor

@martinlingstuyl martinlingstuyl commented Jun 29, 2022

Closes #3457

Updates spo tenant appcatalog add reference to deprecated spo site classic add command.

Remark

We're currently still creating a classic AppCatalog site. This is OK, but SharePoint itself creates a modern AppCatalog site these days. This means SharePoint is much faster creating it. We might look into that using another issue. Maybe we can update the creation as well.

Copy link
Copy Markdown
Member

@Adam-it Adam-it left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

all good.
what maybe good to know is that we should not merge this PR until the
#3456
and
#3450
gets merged 🤔. If you agree I would put it for draft just until those two will be merged so that this PR won't get merged first by mistake

... BTW. you rock 🤩

@Adam-it Adam-it marked this pull request as draft July 8, 2022 22:11
@martinlingstuyl
Copy link
Copy Markdown
Contributor Author

Hi @Adam-it, no this PR can be merged already. spo site add and spo site classic add where consolidated some time ago by @garrytrinder.

@martinlingstuyl martinlingstuyl marked this pull request as ready for review July 10, 2022 11:18
@Adam-it Adam-it self-assigned this Jul 17, 2022
Copy link
Copy Markdown
Member

@Adam-it Adam-it left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please see my comment 👍

Comment thread src/m365/spo/commands/tenant/tenant-appcatalog-add.ts
@Adam-it Adam-it marked this pull request as draft July 17, 2022 21:24
@martinlingstuyl martinlingstuyl force-pushed the updateDeprecatedReference branch from f714b24 to 3e5b145 Compare July 18, 2022 20:20
@martinlingstuyl martinlingstuyl marked this pull request as ready for review July 18, 2022 20:22
@martinlingstuyl martinlingstuyl force-pushed the updateDeprecatedReference branch from 3e5b145 to 0286032 Compare July 18, 2022 20:29
@martinlingstuyl
Copy link
Copy Markdown
Contributor Author

I also fixed an issue that originated in the default output being json these days. Nobody must have executed the command in the meantime @Adam-it 😂

@Adam-it
Copy link
Copy Markdown
Member

Adam-it commented Jul 19, 2022

I also fixed an issue that originated in the default output being json these days. Nobody must have executed the command in the meantime @Adam-it 😂

hmm.. that could be one of the reasons I had a hard time testing this command locally 😅. Thanks for investigating this and the additional fixes 👍 you rock 🤩.
I will give a check on my next "PR/merge session" coming up tomorrow I hope 😉

Copy link
Copy Markdown
Member

@Adam-it Adam-it left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅👍🤩

@Adam-it Adam-it removed their assignment Jul 19, 2022
@Adam-it
Copy link
Copy Markdown
Member

Adam-it commented Jul 19, 2022

you rock 🤩merged manually 👍

@Adam-it Adam-it closed this Jul 19, 2022
@martinlingstuyl martinlingstuyl deleted the updateDeprecatedReference branch July 20, 2022 05:18
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.

Move reference of spo tenant appcatalog add to deprecated spo site classic add command

2 participants