chore: update docusaurus

[SimenB]

Summary

Closes #12432

Test plan

Click around deployment preview

[SimenB]

This fails since we haven't done https://docusaurus.io/blog/2021/11/21/algolia-docsearch-migration.

@slorber how do we solve this? 😅 I've never gotten an email from Algolia (I'm probably not a "site owner" tho)

[slorber]

@shortcuts can tell who's the site owner and eventually set a new one.

I guess it's better if Meta owns the Algolia account for Jest? 🤷‍♂️
(you can be added as a member in the admin dashboard)

Not sure who to contact, maybe @zpao @hramos ?

[shortcuts]

Hey, checking..

Edit: on my side @vjeux is the owner of the Jest account, but the invite expired (7 days validity). Let me know if I should add anyone else, you'll then be able to manage team members yourself

[slorber]

@SimenB I have access and invited you

Just use this on the config:

indexName: jest-v2
appId: HP439UUSOL
apiKey: e5e670fd16f8f17caada79d6b0931682

Similarly the archived deployment will be fixed in #12543

[SimenB]

Thanks! Makes sense for some Meta employee (or them as an org) being admin, yes 🙂

[SimenB]

some other error... I'll look into it tomorrow

[slorber]

@SimenB this is related to a breaking change we did on the TOC component: now the TOC structure is "flat" instead of being a tree: facebook/docusaurus#6729

When running locally on those reported URLs you can see a better console error.

---

Some of the Jest pages are using inline TOCs with some TOC filtering and should be updated accordingly.

It's not possible anymore to access .children on top items

- <TOCInline toc={toc[toc.length - 1].children}/>
+ <TOCInline toc={toc.slice(2)} />

---

cc @Josh-Cena maybe we should have a clearer error message? I don't think this would impact a lot of users though 🤷‍♂️

[Josh-Cena]

maybe we should have a clearer error message?

There isn't much we can do, since the .children access happens on the client side, before it is even passed to any Docusaurus code... The only way is to have something like:

const toc = [
  {
     get children() { throw new Error("children doesn't exist"); }
  },
];

which is just weird...

[slorber]

was more thinking about throwing if toc is undefined, but probably useless

Let's see if someone else come up with a similar problem

[SimenB]

Aha, thanks @slorber!

[github-actions]

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.
Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions.