Skip to content

Conversation

@SamyPesse
Copy link
Member

This PR aims to use static routes for VA and adaptive content. The authentication is done in the middleware already with getPublishedContentByUrl, but now the site data from it are passed in the URL using a Rison encoded route parameter.

Since the apiToken from getPublishedContentByUrl are now stable over period of time, it should not cause cache misses.

@changeset-bot
Copy link

changeset-bot bot commented Mar 19, 2025

⚠️ No Changeset found

Latest commit: e58b714

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions
Copy link
Contributor

github-actions bot commented Mar 19, 2025

@argos-ci
Copy link

argos-ci bot commented Mar 19, 2025

The latest updates on your projects. Learn more about Argos notifications ↗︎

Build Status Details Updated (UTC)
customers-v1 (Inspect) ✅ No changes detected - Mar 20, 2025, 1:53 PM
customers-v2 (Inspect) ✅ No changes detected - Mar 20, 2025, 1:55 PM
default (Inspect) 👍 Changes approved 2 changed Mar 20, 2025, 1:56 PM
v2-vercel (Inspect) ✅ No changes detected - Mar 20, 2025, 1:58 PM

@SamyPesse SamyPesse changed the title v2: experiment with static route for VA / adaptive content v2: use static route for VA / adaptive content Mar 20, 2025
@SamyPesse SamyPesse marked this pull request as ready for review March 20, 2025 13:49
@SamyPesse SamyPesse merged commit 4e5d6c7 into main Mar 20, 2025
18 checks passed
@SamyPesse SamyPesse deleted the v2-experiemnt-json-route branch March 20, 2025 14:41
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.

3 participants