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(v2): support rendering of singular tabs #3823

Merged
merged 3 commits into from
Nov 26, 2020
Merged

Conversation

aeneasr
Copy link
Contributor

@aeneasr aeneasr commented Nov 26, 2020

Closes #3822

Motivation

(Write your motivation here.)

Have you read the Contributing Guidelines on pull requests?

(Write your answer here.)

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work. Bonus points for screenshots and videos!)

Related PRs

(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

@facebook-github-bot facebook-github-bot added the CLA Signed Signed Facebook CLA label Nov 26, 2020
@netlify
Copy link

netlify bot commented Nov 26, 2020

Deploy preview for docusaurus-2 ready!

Built with commit f6ea0d5

https://deploy-preview-3823--docusaurus-2.netlify.app

@aeneasr
Copy link
Contributor Author

aeneasr commented Nov 26, 2020

Looks like a tab definition is failing, not sur ehow to fix that - but maybe this can give an idea of what needs to be done!

@slorber
Copy link
Collaborator

slorber commented Nov 26, 2020

added a fix to your branch, it should work now

@slorber slorber added the pr: bug fix This PR fixes a bug in a past release. label Nov 26, 2020
@slorber slorber merged commit fc4b756 into facebook:master Nov 26, 2020
@aeneasr
Copy link
Contributor Author

aeneasr commented Nov 26, 2020

Thank you! Given that this broke all builds for us, is there any chance to have this released in the near future?

@aeneasr aeneasr deleted the patch-2 branch November 26, 2020 16:51
@slorber
Copy link
Collaborator

slorber commented Nov 26, 2020

@aeneasr in the process of adding canary releases, so that users can run on master code easily, so you may be able to do so very soon if you urgently need an unreleased fix

@aeneasr
Copy link
Contributor Author

aeneasr commented Nov 26, 2020

wow that's awesome!

@slorber
Copy link
Collaborator

slorber commented Nov 26, 2020

@aeneasr a canary dist tag has been published, you can try it and let me know if it works for you?

This fix should be available

@aeneasr
Copy link
Contributor Author

aeneasr commented Nov 27, 2020

Yes, that works perfectly - thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed Signed Facebook CLA pr: bug fix This PR fixes a bug in a past release.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[v2] Tabs fail to render when they have only one child
3 participants