Merge pull request #480 from Shopify/v3_authenticate_public

V3 authenticate public

.changeset/spotty-teachers-enjoy.md

@@ -0,0 +1,29 @@
+---
+'@shopify/shopify-app-remix': minor
+---
+
+Added v3_authenticatePublic feature flag to remove `authenticate.public(request)`.
+
+Apps can opt in to the new future at any time, so this is not a breaking change until version 3.
+
+  <details>
+    <summary>See an example</summary>
+
+Without the `v3_authenticatePublic` future flag the deprecated `authenticate.public(request)` is supported:
+
+```ts
+await authenticate.public.checkout(request);
+await authenticate.public.appProxy(request);
+
+// Deprecated.  Use authenticate.public.checkout(request) instead
+await authenticate.public(request);
+```
+
+With the `v3_authenticatePublic` future flag enabled the deprecated `authenticate.public(request)` is not supported:
+
+```ts
+await authenticate.public.checkout(request);
+await authenticate.public.appProxy(request);
+```
+
+  </details>

packages/shopify-app-remix/src/server/__test-helpers/test-config.ts

@@ -17,6 +17,10 @@ type TestConfig<Overrides extends Partial<AppConfigArg>> =
     Overrides & {
       // Create an object with all future flags defaulted to active to ensure our tests are updated when we introduce new flags
       future: {
+        v3_authenticatePublic: DefaultedFutureFlag<
+          Overrides,
+          'v3_authenticatePublic'
+        >;
         v3_webhookAdminContext: DefaultedFutureFlag<
           Overrides,
           'v3_webhookAdminContext'
@@ -42,6 +46,7 @@ export function testConfig<Overrides extends Partial<AppConfigArg>>(
     ...overrides,
     future: {
       v3_webhookAdminContext: true,
+      v3_authenticatePublic: true,
       ...(overrides.future as Overrides['future']),
     },
   };