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.

Sign up for GitHub

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

Jump to bottom

Move function from the graphql_register_types action hook #162

Merged
merged 4 commits into from Jun 30, 2023
Merged

Move function from the graphql_register_types action hook #162

merged 4 commits into from Jun 30, 2023

Conversation

renatonascalves
Copy link
Contributor

resolves #161

@renatonascalves renatonascalves changed the title Move a function from the graphql_register_types action hook Move function from the graphql_register_types action hook Jun 28, 2023
@renatonascalves
Copy link
Contributor Author

I wonder if this function's name should be changed as well. 🤔 Into something unique to the plugin.

@ashhitch
Copy link
Owner

@renatonascalves thanks for this! Sound sensible to rename it if its causing issues.

@renatonascalves
Copy link
Contributor Author

@ashhitch Done! :)

@renatonascalves
Copy link
Contributor Author

@ashhitch Bumping this! 🙏🏾

@ashhitch ashhitch merged commit 873e7d3 into ashhitch:master Jun 30, 2023
@ashhitch
Copy link
Owner

All merged in I have another branch with another bug fix will get them released together.
Thanks 👍

@renatonascalves renatonascalves deleted the feature/hot-fix branch June 30, 2023 18:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Cannot redeclare get_post_type_graphql_fields() previously declared
2 participants
@renatonascalves @ashhitch