Link overwrites redirect urls

[alvaro-shopify]

WHY are these changes introduced?

Fixes https://github.com/Shopify/develop-app-foundations/issues/886

WHAT is this pull request doing?

• In the link command when local configuration is merged the remote one, a replace with remote strategy is used instead of union, for any array property

How to test your changes?

• Run dev to automatically update app URLs
• Run link, the redirect_urls property content is overwritten with the remote one

Post-release steps

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
• I've made sure that any changes to dev or deploy have been reflected in the internal flowchart.

[github-actions]

Thanks for your contribution!

Depending on what you are working on, you may want to request a review from a Shopify team:

• Themes: @shopify/advanced-edits
• UI extensions: @shopify/ui-extensions-cli
  • Checkout UI extensions: @shopify/checkout-ui-extensions-api-stewardship
• Hydrogen: @shopify/hydrogen
• Other: @shopify/cli-foundations

[github-actions]

Coverage report

St.❔	Category	Percentage	Covered / Total
🟡	Statements	73.96% (-0.02% 🔻)	6666/9013
🟡	Branches	71.08% (-0.04% 🔻)	3269/4599
🟡	Functions	73.49% (+0.01% 🔼)	1744/2373
🟡	Lines	75.02% (-0.03% 🔻)	6288/8382

Show files with reduced coverage 🔻

St.❔	File	Statements	Branches	Functions	Lines
🟢	... / ConcurrentOutput.tsx	97.62% (-2.38% 🔻)	75% (-8.33% 🔻)	100%	97.44% (-2.56% 🔻)

Test suite run success

1594 tests passing in 743 suites.

Report generated by 🧪jest coverage report action from f3210b8

[alexanderMontague]

Comment about the test, but tophat works as expected 👍