-
-
Notifications
You must be signed in to change notification settings - Fork 110
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
V2 - New preset v2-mst-aptd-at-lcz-sty
with Airtable replacing GraphCMS (based on v2-mst-aptd-gcms-lcz-sty
)
#86
base: v2-mst-aptd-gcms-lcz-sty
Are you sure you want to change the base?
Conversation
[GitHub Actions] |
5f8b8f7
to
5d1274d
Compare
[GitHub Actions] |
1e90bfc
to
b5e7a8c
Compare
[GitHub Actions] |
[GitHub Actions] |
[GitHub Actions] |
[GitHub Actions] |
[GitHub Actions] |
[GitHub Actions] |
[GitHub Actions] |
[GitHub Actions] |
c571f31
to
adca1e7
Compare
[GitHub Actions] |
[GitHub Actions] |
[GitHub Actions] |
[GitHub Actions] |
[GitHub Actions] |
[GitHub Actions] |
[GitHub Actions] |
[GitHub Actions] |
[GitHub Actions] |
[GitHub Actions] |
[GitHub Actions] |
[GitHub Actions] |
[GitHub Actions] |
…try/init"' has no default export."
* Re-export Sentry from init (DX) and call init from error endpoint to see if it fixes the issue with flush() timeout * Improve sentry init documentation + rollback to timeout 2s + auto-init Sentry when using configureReq * Implement better error handling on _error page (enrich Sentry metadata from req object) * Fix 404 error handling and reporting to Sentry * Improve documentation about error handling * Upgrade Sentry to latest (minor)
✅ Deployment SUCCESS |
✅ E2E tests SUCCESS for commit 1542318 previously deployed Storybook static site at https://nrn-v2-mst-aptd-at-lcz-sty-storybook-4cccxlccg-unly-oss.vercel.app |
✅ Deployment SUCCESS |
✅ E2E tests SUCCESS for commit 003ea51 previously deployed Storybook static site at https://nrn-v2-mst-aptd-at-lcz-sty-storybook-8jmdx5l0i-unly-oss.vercel.app |
…API (which now returns width/height for all attachments, which broke our dynamic property lookup by overriding our own width/height props)
2470dce
to
3103cab
Compare
❌ Deployment FAILED |
Demos
Status
This PR/branch is stable and has been added as the new official/default NRN preset.
Notes
This PR is not meant to be merged, but to provide a diff against the base branch (which is a preset using GraphCMS)
Objectives
The goal is to replace GraphQL API by Airtable API, for several reasons:
Base preset: v2-mst-aptd-gcms-lcz-sty
New preset name: v2-mst-aptd-at-lcz-sty
Roadmap
AirtableAsset
component)