Skip to content

Add import extensions functionality on deploy #6136

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

Draft
wants to merge 1 commit into
base: fix-management-experience
Choose a base branch
from

Conversation

gonzaloriestra
Copy link
Contributor

@gonzaloriestra gonzaloriestra commented Jul 21, 2025

WHY are these changes introduced?

Fixes https://github.com/shop/issues-develop/issues/469

WHAT is this pull request doing?

  • When running deploy with Partners API, it suggests to optionally migrate dashboard extensions:
prompt-partners
  • When running deploy with App Management API, it requires to migrate/remove dashboard extensions to continue:
Monosnap node 2025-07-22 12-24-24

How to test your changes?

  • shopify app deploy

To add a dashboard managed extension in production, this flag can be enabled.

Measuring impact

How do we know this change was effective? Please choose one:

  • n/a - this doesn't need measurement, e.g. a linting rule or a bug-fix
  • Existing analytics will cater for this addition
  • PR includes analytics changes to measure impact

Checklist

  • I've considered possible cross-platform impacts (Mac, Linux, Windows)
  • I've considered possible documentation changes

Copy link
Contributor Author

gonzaloriestra commented Jul 21, 2025

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

Copy link
Contributor

github-actions bot commented Jul 21, 2025

Coverage report

St.
Category Percentage Covered / Total
🟡 Statements
78.01% (+0.03% 🔼)
13048/16726
🟡 Branches
72.16% (+0.04% 🔼)
6367/8823
🟡 Functions
78.17% (-0.01% 🔻)
3383/4328
🟡 Lines
78.4% (+0.03% 🔼)
12353/15756
Show files with reduced coverage 🔻
St.
File Statements Branches Functions Lines
🟢
... / fetch-extensions.ts
100%
86.67% (-13.33% 🔻)
100% 100%
🟡
... / import-extensions.ts
81.08% (-6.8% 🔻)
66.67% (-4.76% 🔻)
72.73% (-16.16% 🔻)
79.41% (-7.25% 🔻)
🟢
... / ConcurrentOutput.tsx
98.36% (-1.64% 🔻)
92% (-4% 🔻)
100%
98.33% (-1.67% 🔻)

Test suite run success

3074 tests passing in 1319 suites.

Report generated by 🧪jest coverage report action from ecc3b74

@gonzaloriestra gonzaloriestra force-pushed the import-extensions-on-deploy branch 6 times, most recently from 16ad951 to d50edc7 Compare July 22, 2025 11:12
@gonzaloriestra gonzaloriestra force-pushed the refactor-import-extensions branch from 6fc3c69 to ee5ba96 Compare July 23, 2025 09:57
@gonzaloriestra gonzaloriestra force-pushed the import-extensions-on-deploy branch from d50edc7 to 33bfff1 Compare July 23, 2025 09:57
Copy link
Contributor

Unused types (1)

Filename types
packages/app/src/cli/services/fetch-extensions.ts GetExtensionsOptions

@gonzaloriestra gonzaloriestra force-pushed the import-extensions-on-deploy branch from 33bfff1 to 3ccbc1f Compare July 23, 2025 10:52
@gonzaloriestra gonzaloriestra changed the base branch from refactor-import-extensions to graphite-base/6136 July 23, 2025 11:23
@gonzaloriestra gonzaloriestra force-pushed the import-extensions-on-deploy branch from 3ccbc1f to c77d13f Compare July 23, 2025 11:24
@gonzaloriestra gonzaloriestra changed the base branch from graphite-base/6136 to fix-management-experience July 23, 2025 11:24
@gonzaloriestra gonzaloriestra force-pushed the import-extensions-on-deploy branch from c77d13f to 0710e55 Compare July 23, 2025 11:27
@gonzaloriestra gonzaloriestra force-pushed the fix-management-experience branch from ee1c795 to 5d44a22 Compare July 23, 2025 11:27
@gonzaloriestra gonzaloriestra force-pushed the import-extensions-on-deploy branch from 0710e55 to c9034e3 Compare July 23, 2025 13:46
@gonzaloriestra gonzaloriestra force-pushed the fix-management-experience branch from 5d44a22 to a990e97 Compare July 23, 2025 13:46
@gonzaloriestra gonzaloriestra force-pushed the import-extensions-on-deploy branch from c9034e3 to 3cc6524 Compare July 24, 2025 14:35
@gonzaloriestra gonzaloriestra force-pushed the import-extensions-on-deploy branch from 3cc6524 to 453f4ea Compare July 24, 2025 15:00
@gonzaloriestra gonzaloriestra force-pushed the fix-management-experience branch from a990e97 to 2ec9c8d Compare July 24, 2025 15:00
Copy link
Contributor

Differences in type declarations

We detected differences in the type declarations generated by Typescript for this branch compared to the baseline ('main' branch). Please, review them to ensure they are backward-compatible. Here are some important things to keep in mind:

  • Some seemingly private modules might be re-exported through public modules.
  • If the branch is behind main you might see odd diffs, rebase main into this branch.

New type declarations

We found no new type declarations in this PR

Existing type declarations

packages/cli-kit/dist/public/common/version.d.ts
@@ -1 +1 @@
-export declare const CLI_KIT_VERSION = "3.83.0";
\ No newline at end of file
+export declare const CLI_KIT_VERSION = "3.82.0";
\ No newline at end of file

1 similar comment
Copy link
Contributor

Differences in type declarations

We detected differences in the type declarations generated by Typescript for this branch compared to the baseline ('main' branch). Please, review them to ensure they are backward-compatible. Here are some important things to keep in mind:

  • Some seemingly private modules might be re-exported through public modules.
  • If the branch is behind main you might see odd diffs, rebase main into this branch.

New type declarations

We found no new type declarations in this PR

Existing type declarations

packages/cli-kit/dist/public/common/version.d.ts
@@ -1 +1 @@
-export declare const CLI_KIT_VERSION = "3.83.0";
\ No newline at end of file
+export declare const CLI_KIT_VERSION = "3.82.0";
\ No newline at end of file

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant