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

Fix: createTerm issue with custom hierarchical taxonomy #51

Merged
merged 1 commit into from
Apr 11, 2022
Merged

Conversation

dinhtungdu
Copy link
Contributor

@dinhtungdu dinhtungdu commented Apr 6, 2022

Description of the Change

This PR fix the issue with custom hierarchical taxonomy when using cy.createTerm to create the first taxonomy. The parent dropdown exists in HTML but isn't visible. This causes the timeout issue as described in the #50.

Closes #50

Checklist:

  • I have read the CONTRIBUTING document.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my change.
  • All new and existing tests passed.

Changelog Entry

Credits

Props @dinhtungdu

@dinhtungdu dinhtungdu marked this pull request as ready for review April 6, 2022 05:10
@dinhtungdu dinhtungdu added type:bug Something isn't working. needs:code-review This requires code review. labels Apr 6, 2022
@dinhtungdu dinhtungdu requested a review from cadic April 6, 2022 05:16
Copy link
Contributor

@cadic cadic left a comment

Choose a reason for hiding this comment

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

Great catch, @dinhtungdu and thank you for the fix!

The current PR is LGTM!

I have created separate issue to cover this with tests #52

@cadic cadic merged commit b7086c6 into trunk Apr 11, 2022
@cadic cadic deleted the fix/50 branch April 11, 2022 09:56
@cadic cadic added the needs:tests This requires tests. label Apr 11, 2022
github-actions bot pushed a commit that referenced this pull request Apr 11, 2022
Fix: createTerm issue with custom hierarchical taxonomy
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs:code-review This requires code review. needs:tests This requires tests. type:bug Something isn't working.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

createTerm doesn't work with custom hierarchical taxonomy
2 participants