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: corrected anchor tags href in Playground sidebar #859

Closed
wants to merge 2 commits into from
Closed

fix: corrected anchor tags href in Playground sidebar #859

wants to merge 2 commits into from

Conversation

Santhoshmani1
Copy link

@Santhoshmani1 Santhoshmani1 commented Dec 12, 2023

Description

Changes proposed in this pull request:

  • Fixed anchor links in playground Sidebar by adding id to the operations.

Related issue(s)

Fixes

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Welcome to AsyncAPI. Thanks a lot for creating your first pull request. Please check out our contributors guide useful for opening a pull request.
Keep in mind there are also other channels you can use to interact with AsyncAPI community. For more details check out this issue.

@Santhoshmani1 Santhoshmani1 changed the title fix: corrected anchor tags href in Playground sidebar chore : corrected anchor tags href in Playground sidebar Dec 12, 2023
@Santhoshmani1 Santhoshmani1 changed the title chore : corrected anchor tags href in Playground sidebar fix: corrected anchor tags href in Playground sidebar Dec 12, 2023
@BTOdell
Copy link

BTOdell commented Dec 14, 2023

This doesn't appear to fix the problem correctly. You're adding a new anchor instead of fixing the existing anchor with the format: #operation-{label}-{channel}.

@derberg
Copy link
Member

derberg commented Dec 18, 2023

@Santhoshmani1 please have a look at comment from @BTOdell

Copy link

sonarcloud bot commented Dec 18, 2023

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@Santhoshmani1
Copy link
Author

Async.API.playground.links.corrected.mp4

I have changed the href attribute of sidebar links and added the id attribute to match in the operations. It's working correctly as expected and fixes the problem.

@@ -360,7 +360,7 @@ const OperationsList: React.FunctionComponent = () => {
interface OperationItemProps {
channelName: string;
summary: string;
kind: 'publish' | 'subscribe';
kind: 'subscribe' | 'publish';
Copy link
Member

Choose a reason for hiding this comment

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

why is this change needed?

library/src/containers/Sidebar/Sidebar.tsx Show resolved Hide resolved
@kennethaasan
Copy link
Contributor

@Santhoshmani1, I just wanted to say thank you for the contribution 🚀

@Santhoshmani1 Santhoshmani1 closed this by deleting the head repository Jan 2, 2024
@derberg
Copy link
Member

derberg commented Jan 4, 2024

@Santhoshmani1 you do not want to continue with this one?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants