feat: add error message when project not aligned to spaceId

[lilbitner]

Purpose

The purpose of this PR is to add an error message when the user selects a project that is not configured to be in the space where the current app is installed/being installed.

Approach

I added the below error message on render of the page AND when the user changes project, and the selected project contains an incompatible CONTENTFUL_SPACE_ID variable. The error lingers until the user changes to a valid project, but they are still able to install and save changes.

Testing steps

Within this space, I have created one Vercel project that is compatible with the space: nextjs-lb-4. The rest of the projects are not compatible, so you should see warnings for all other project selections.

[netlify]

✅ Deploy Preview for ecommerce-app-base-components canceled.

Name	Link
🔨 Latest commit	334ac4a
🔍 Latest deploy log	https://app.netlify.com/sites/ecommerce-app-base-components/deploys/66394acefddf9d00086258af

[netlify]

✅ Deploy Preview for ecommerce-app-base-components ready!

Name	Link
🔨 Latest commit	45f72c8
🔍 Latest deploy log	https://app.netlify.com/sites/ecommerce-app-base-components/deploys/663a64e4c77f410008c188cd
😎 Deploy Preview	https://deploy-preview-7587--ecommerce-app-base-components.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[jsdalton]

Code looks good, just one or two small suggestions.

A few questions about the error message:

• Should this be a warning? I feel like we don't have enough confidence that it's wrong to show a true error.
• Probably would be nice to show the value of that env var in the message? I guess it might help allay confusion.

[lilbitner]

Code looks good, just one or two small suggestions.

A few questions about the error message:

• Should this be a warning? I feel like we don't have enough confidence that it's wrong to show a true error.
• Probably would be nice to show the value of that env var in the message? I guess it might help allay confusion.

Yeah good questions!

1. In my opinion, we do have a solid amount of confidence that there is a mismatch of spaceIds here and that the app will not work without this being fixed (per my understanding, I could be missing something). Along with this, there really isn't precedent in our design system for a "warning" type for the Select component. It is fairly binary (valid or invalid). I could certainly create custom component to add here instead, but I do think it is breaking enough to warrant an error. Honestly might be good feedback for F36 🤔
2. Yeah I also considered this. This adds a bit more complexity to the error handling, but I could do this as part of some follow-up work, to make it easier for the user to adjust the spaceId if they need to 👍

[matthew-gordon]

Looks good and makes sense to me 🤌 ✨