Open dishafaujdar opened 4 days ago
The pull request introduces two main changes: the Calendar
component in components/Calendar.tsx
now includes the h-full
class in its main <div>
, ensuring it occupies the full height of its parent container. Additionally, the link
property of the last bucket object in components/data/buckets.ts
is updated to change the case of "Explorer" to "explorer" in the URL, correcting a potential issue with accessing the Specification Explorer documentation.
File | Change Summary |
---|---|
components/Calendar.tsx | Added h-full class to the main <div> to ensure full height of the parent container. |
components/data/buckets.ts | Updated the link for the "Explorer" bucket from /docs/reference/specification/v3.0.0-Explorer to /docs/reference/specification/v3.0.0-explorer . |
GeneratorPage
component involve CSS modifications that may relate to overall styling practices similar to the addition of the h-full
class in the Calendar
component, indicating a focus on layout consistency.edit-page-config.json
file involves correcting a link, which, while not directly related to the Calendar
component, reflects a broader effort to ensure accurate resource linking and documentation, potentially impacting user experience in a similar context.ready-to-merge
π° In the calendar's embrace, we find,
A height so grand, itβs truly aligned.
The explorer's path, now clear and bright,
With links that guide us, day and night.
Hopping through docs, with joy we cheer,
For every change, brings us near! π
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Built without sensitive environment variables
Name | Link |
---|---|
Latest commit | d2d76521c39ebf737b494647417ac556f571912d |
Latest deploy log | https://app.netlify.com/sites/asyncapi-website/deploys/67499edb43568e0008c99f84 |
Deploy Preview | https://deploy-preview-3428--asyncapi-website.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
All modified and coverable lines are covered by tests :white_check_mark:
Please upload report for BASE (
master@e56289b
). Learn more about missing BASE report.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
β‘οΈ Lighthouse report for the changes in this PR: | Category | Score |
---|---|---|
π΄ Performance | 34 | |
π’ Accessibility | 98 | |
π’ Best practices | 92 | |
π’ SEO | 100 | |
π΄ PWA | 33 |
Lighthouse ran on https://deploy-preview-3428--asyncapi-website.netlify.app/
@dishafaujdar The PR title subject should start with a lowercase character. Rename your PR to fix linting errors.
The browser requests in https://github.com/asyncapi/website/blob/master/components/data/buckets.ts#L96 'Explorer,' but the App Router in https://github.com/asyncapi/website/blob/master/components/layout/DocsLayout.tsx#L92 is looking for 'explorer.' It does not find it and returns an HTTP 404 error in the local environment.
This PR fixes the inconsistency, ensuring the link returns the correct page after clicking on 'Docs / Specification Explorer'.
Summary by CodeRabbit
New Features
Bug Fixes