Skip to content

Conversation

@ryanmitchell
Copy link
Contributor

@ryanmitchell ryanmitchell commented Aug 22, 2025

This PR adds Statamic v6 Support.

To do:

  • Move user config to add-on settings
  • Update field types
  • Create updater that migrates the config
  • Remove existing config file
  • Should API key be in config file?
  • get fieldtypes building

@ryanmitchell ryanmitchell marked this pull request as ready for review August 22, 2025 09:40
@ryanmitchell ryanmitchell requested a review from edalzell August 22, 2025 09:40
Copy link
Collaborator

@edalzell edalzell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I trust you, yolo

@edalzell
Copy link
Collaborator

edalzell commented Nov 2, 2025

Any idea why these tests fail @ryanmitchell ?

@ryanmitchell
Copy link
Contributor Author

ryanmitchell commented Nov 2, 2025

@edalzell tests are passing now

Create updater that migrates the config

This is already in the ServiceProvider - migrateUserToSettings()

Remove existing config file

This is done as part of the same function

Should API key be in config file?

I nearly did this, but I left it so it could be defined in the .env which is what people expect I think.

@edalzell
Copy link
Collaborator

edalzell commented Nov 2, 2025

Create updater that migrates the config

This is already in the ServiceProvider - migrateUserToSettings()

Sure but why in provider and not an updater script?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants