website icon indicating copy to clipboard operation
website copied to clipboard

[BUG] <description> incorrect repo url for 'AsyncAPI format' in cli category on tools page

Open impakshal opened this issue 1 month ago • 10 comments

Describe the bug.

View github button in 'AsyncAPI format' div incorrectly redirects to converter-go repository

Expected behavior

AsyncAPI-format should redirect to correct repository which is https://github.com/thim81/asyncapi-format as correctly listed in the Converters section. (please verify the link)

Screenshots

Image

How to Reproduce

  1. Go to config/tools-manual.json and fix the link this should work and fix the issue

🖥️ Device Information [optional]

  • Operating System (OS):
  • Browser:
  • Browser Version:

👀 Have you checked for similar open issues?

  • [x] I checked and didn't find similar issue

🏢 Have you read the Contributing Guidelines?

Are you willing to work on this issue ?

Yes I am willing to submit a PR!

impakshal avatar Dec 07 '25 09:12 impakshal

If the issue is worthy.. I am more than happy to fix this.. So please allow me to work upon it @princerajpoot20

impakshal avatar Dec 07 '25 09:12 impakshal

Hi, I checked config/tools-manual.json and the repo URL for AsyncAPI-format already points to https://github.com/thim81/asyncapi-format.

This issue appears to be fixed and can be closed. ✅

Dsp023 avatar Dec 09 '25 16:12 Dsp023

Hello @Dsp023.... Thank you for your suggestions There are 2 AsyncApi-format... I am talking about one in CLIs div... Kindly check it and let me know if it still appears good

impakshal avatar Dec 12 '25 11:12 impakshal

AsyncAPI-format should redirect to correct repository which is https://github.com/thim81/asyncapi-format

@impakshal Could you tell me the source of the link you provided (https://github.com/thim81/asyncapi-format )? I mean, where did you get this link from?

princerajpoot20 avatar Dec 13 '25 10:12 princerajpoot20

Hello @princerajpoot20 ...Thanks for your reply.... There are 2 AsyncApi-format divs in tools page One directs to thim81 github repo and other to converter-go. So i thought https://github.com/thim81/asyncapi-format this is genuine link ...if not please provide me the actual repo link. I will change it for both AsyncApi-format

Image Image

impakshal avatar Dec 13 '25 10:12 impakshal

@impakshal So, as you mentioned, there are two divs for the same tool, and from the screenshot you shared, they seem to be identical. Don’t you think we should remove one? Does it make sense to have two identical entries?

princerajpoot20 avatar Dec 13 '25 11:12 princerajpoot20

@princerajpoot20 I went through entire tools page and all divs present... There are duplicate divs present at multiple places that keep on repeating in different categories...So i think it is general idea of page that a feature that falls under 2 or more category is listed under all of them.. Please check it and let me know Also i discussed about duplicate divs in same category at in #4690 (please check the comment i made last week)

impakshal avatar Dec 13 '25 11:12 impakshal

The reason for the duplication is the tools-automated file. There is a workflow that fetches all the data from GitHub using an API call.

At the moment, we can only fix the data present in the tools-manual file. If anything is wrong in the tools-automated file, we cannot fix it at the moment, as the automation will overwrite it. Anything that needs to be fixed in the tools-manual file is the only thing we can address right now.

princerajpoot20 avatar Dec 13 '25 11:12 princerajpoot20

@princerajpoot20 ...So this issue restricts itself to tools-manual file only If the repo link https://github.com/thim81/asyncapi-format is correct then i can update the wrong repo link of one div

impakshal avatar Dec 13 '25 11:12 impakshal

@princerajpoot20 waiting for reply

impakshal avatar Dec 16 '25 13:12 impakshal