feat: add default sourceID configuration fields

[luqven]

Description

• Adds a configuration field for a default source ID
• Updates sources dropdown button to show loading while fetching sources
• Defaults the Dialog to the configured source ID if set, otherwise fetches sources list

Before

• No way to configure a default source
• Loading indicator on sources dropdown was only shown when clicking dropdown
• Hitting "save" with an invalid configuration would persist the changes despite showing a warning or error

After

• Customers can configure a default source
• Loading indicator for sources dropdown visible without clicking
• Trying to save an invalid configuration does not persist the changes

Loading indicator

source-dropdown-loading-indicator.mp4

Default source

source-dropdown-default-source.mp4

API KEY Validation

NOTE: the source ID is hidden in the video but not when you clone or preview this branch. This is on purpose. I'm only hiding the source ID on the video out of extreme caution.

default-source-validation.mp4

Steps to test

1. Clone, npm install, and pnpm run start
2. Go to Contentful development space
3. Opent the content with the localhost field
4. Remove any previously selected image
5. Refresh the page
6. To go Apps --> Installed Apps --> imgix development --> configure
7. Add the sourceID of your choosing as the default sourceID
8. See that API key validation succeeds/fails if associated/not associated with said source
9. Go back to the previous content entry
10. Add an image
11. See that the modal defaults to the configured source ID

Checklist

• Read the contributing guidelines.
• Each commit follows the Conventional Commit spec format.
• Update the readme (if applicable).
• Update or add any necessary API documentation (if applicable)
• All existing unit tests are still passing (if applicable).

• Add some steps so we can test your bug fix or feature (if applicable).
• Add new passing unit tests to cover the code introduced by your PR (if applicable).
• Any breaking changes are specified on the commit on which they are introduced with BREAKING CHANGE in the body of the commit.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
contentful	🛑 Canceled (Inspect)			Jun 21, 2024 0:09am
contentful-staging	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jun 21, 2024 0:09am

[marco-salcedo]

Left a non-blocking comment, LGTM 🚀

[marco-salcedo]

suggestion (non-blocking): I almost called out that the comparison below was in invalid, but seeing what previouslySelectedSource actually is, I see that its a direct comparison.

I think we should rename this variable previouslySelectedSourceId and update where needed.

Suggested change

	const previouslySelectedSource =
	invocationParams?.selectedImage?.selectedSource?.id ||
	installationParams.sourceID;
	const previouslySelectedSourceId =
	invocationParams?.selectedImage?.selectedSource?.id ||
	installationParams.sourceID;

[luqven]

comment:

I can see why that's confusing 😅 , I can update this variable name to make this easier to grok 👍🏼 .