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

Deprecate Old publishedOn and ownedBy GraphQL Fields (Instead of Removing Them) and Make New Meta GraphQL Fields Nullable #4156

Merged
merged 82 commits into from
Jun 7, 2024

Conversation

adrians5j
Copy link
Member

@adrians5j adrians5j commented Jun 6, 2024

Changes

With this PR, we're bringing back the removed publishedOn and ownedBy GraphQL fields, just so existing users have a fallback when upgrading to 5.39.

We've also made the new meta GraphQL fields nullable. This makes it possible for users to use the Admin app even while their system is being data migrated.

How Has This Been Tested?

Manually.

Documentation

Changelog.

adrians5j and others added 30 commits May 15, 2024 13:22
For now, we're removing the logger just so redundant logs are no longer printed in CloudWatch. In the near future, we'll have our own Pino logger that we'll pass to Fastify.
…webiny-js into adrian/migration-script-using-mt
@adrians5j adrians5j marked this pull request as ready for review June 7, 2024 04:42
@adrians5j adrians5j changed the base branch from adrian/migration-script-using-mt-2 to dev June 7, 2024 12:15
@adrians5j adrians5j merged commit 49e3958 into dev Jun 7, 2024
84 checks passed
@adrians5j adrians5j deleted the adrian/deprecated-gql-meta-fields branch July 5, 2024 07:00
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.

2 participants