Skip to content

Change type of tagId from String to Int#1779

Closed
rubas wants to merge 1 commit intowp-graphql:developfrom
rubas:patch-2
Closed

Change type of tagId from String to Int#1779
rubas wants to merge 1 commit intowp-graphql:developfrom
rubas:patch-2

Conversation

@rubas
Copy link
Copy Markdown

@rubas rubas commented Mar 15, 2021

What does this implement/fix? Explain your changes.

Change type of tagId from String to Int.

Probably just a typo for tagId.

Any other comments?

I suggest changing all the ..Id to the type ID to be consistent.

Probably just a typo for Tag.

Personally I would suggest changing all of the ..Id to the type ID to be consistent.
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 79.11% when pulling 93ad147 on rubas:patch-2 into dcfdc6b on wp-graphql:develop.

@justlevine justlevine added compat: breaking change This is a breaking change to existing functionality needs: tests Tests should be added to be sure this works as expected labels May 13, 2022
@stale
Copy link
Copy Markdown

stale bot commented Aug 2, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Aug 2, 2022
@stale
Copy link
Copy Markdown

stale bot commented Sep 1, 2022

This issue has been automatically closed because it has not had recent activity. If you believe this issue is still valid, please open a new issue and mark this as a related issue.

@stale stale bot closed this Sep 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

compat: breaking change This is a breaking change to existing functionality needs: tests Tests should be added to be sure this works as expected

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants