Skip to content
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

Remove deprecated configuration and package exports #9083

Merged
merged 16 commits into from
Apr 4, 2024
Merged

Remove deprecated configuration and package exports #9083

merged 16 commits into from
Apr 4, 2024

Conversation

dcousens
Copy link
Member

@dcousens dcousens commented Apr 4, 2024

This pull request removes a significant number of the deprecated configuration options and capabilities, with replacements suggested inline to each of the changes.

Some changes do not have replacements, namely because we think they were bad patterns to begin with, examples of this include:

  • context.exitSudo()
  • config.server.healthCheck, please use your own route within extendExpressApp, this is too broad a topic for Keystone to be prescriptive about
  • context.experimental.initialisedLists (if you need this, you can use context.__internal.lists as a replacement, but like experimental, it is not considered stable)

@dcousens dcousens force-pushed the major2 branch 3 times, most recently from 2e6ba5c to 4101ded Compare April 4, 2024 06:46
@keystonejs keystonejs deleted a comment from codesandbox-ci bot Apr 4, 2024
@dcousens dcousens force-pushed the major2 branch 2 times, most recently from fe43f81 to d43a14b Compare April 4, 2024 12:18
@dcousens dcousens force-pushed the major2 branch 4 times, most recently from 2dc9a15 to ed1cfb8 Compare April 4, 2024 13:31
@dcousens dcousens force-pushed the major2 branch 2 times, most recently from e6196a7 to 27bdc33 Compare April 4, 2024 14:15
@dcousens dcousens merged commit d609c6b into main Apr 4, 2024
43 checks passed
@dcousens dcousens deleted the major2 branch April 4, 2024 14:40
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.

1 participant