-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
update Docusaurus and other deps, fix broken styles #3059
Conversation
✅ Deploy Preview for react-native ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
✅ Deploy Preview for react-native ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
@@ -41,7 +41,7 @@ To request access to location, you need to add the following line to your app's | |||
Android API >= 18 Positions will also contain a `mocked` boolean to indicate if position was created from a mock provider. | |||
|
|||
<p> | |||
Android API >= 23 Requires an additional step to check for, and request | |||
Android API >= 23 Requires an additional step to check for, and request |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is this needed?
I think this looks good! Are there particular places we should look for breakages? I guess, also should we confirm that GA tracking is working (tbh not sure how to do that) |
Generally every layout have at least one issue, sidebars too, only home page looks correct after upgrade, so it's hard to point on the specific thing. I usually test few pages, with focus on the origin diversity:
The fix has been deployed yesterday in separate PR, so you probably should see a data spike on the GA dashboard now, if everything is working correctly. |
Oh actually I do think I found an issue -- I thought it was a general issue but it's not reproing off prod: #3061 |
I have fixed the mobile menu hover issue and the deploy preview should be updated. I have set the hover color, to one which we use for highlighting the active page in the menu. Please let me know, if now it looks fine for you. |
Why & how
Superseeds:
This PR updates Docusaurus to latest beta (
beta.18
) and most of the other dependencies.Due to changes in Docusaurus code, I had to adjust the config and add a bunch of fixes to our custom styles, which become broken after the update. I also had to duplicate the GA move in the config, otherwise the website won't start.
Test plan
I have run the updated version of the website on
localhost
and I used the currently shipped website as a 1:1 comparison, so the style issues was easier to spot, however, there still might be small display problems which I was not able to catch. If you see any regression in preview deploy, please let me know!