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

New links get the same address as peeringdb #145

Closed
pcfriek1987 opened this issue Jun 1, 2021 · 6 comments
Closed

New links get the same address as peeringdb #145

pcfriek1987 opened this issue Jun 1, 2021 · 6 comments
Labels
accepted Issue is confirmed and will be worked on bug Confirmed bug or unexpected behavior fixed-in-next-release Issue has already been fixed in the next release

Comments

@pcfriek1987
Copy link

I added the peeringdb link on the left which works fine, but also tried adding a second link on the right, which still has the peeringdb link even though it has a different value.

links:
- order: 0
show_icon: true
side: left
title: PeeringDB
url: https://www.peeringdb.com/asn/{primary_asn}
- order: 5
side: right
title: Website
url: https://google.com

I replaced the website with a generic one since this is my private github account :)

@thatmattlove
Copy link
Owner

Hmm, I'm not able to reproduce this. My config:

web:
    links:
        - title: PeeringDB
          order: 2
          url: https://peeringdb.com
        - title: Another Link
          order: 1
          url: https://www.google.com
          side: right
        - title: Yet Another Link
          order: 3
          url: https://www.cloudflare.com
          show_icon: false

When I have that in my config and start hyperglass (with --build-ui or -b), I see the links with the correct URLs. Can you try running hyperglass build-ui manually and see if that makes a difference?

@thatmattlove thatmattlove added needs-info Issue lacks appropriate detail or requires additional information possible-bug Something isn't working labels Jun 2, 2021
@pcfriek1987
Copy link
Author

pcfriek1987 commented Jun 2, 2021

I can see it correctly in the build ui but even after a manual build it still goes to the wrong link.

[{'order': 0, 'show_icon': True, 'side': 'left', 'title': 'PeeringDB', 'url': 'https://www.peeringdb.com/asn/{primary_asn}'}, {'title': 'NFOrce Website', 'order': 5, 'side': 'right', 'url': 'https://www.nforce.com/'}]

If you want to see it as well, https://lg.nforce.com/

Nvm seeing the website here, guess it's easier like this.

@thatmattlove
Copy link
Owner

Is this still happening for you? Even when I go to your LG, I see the correct links:

Screen Shot 2021-07-03 at 10 15 10 PM

Screen Shot 2021-07-03 at 10 15 18 PM

@pcfriek1987
Copy link
Author

I actually still do indeed, but it seems to be a firefox issue now that I can see as in Chrome it displays them correctly.

@thatmattlove thatmattlove reopened this Sep 12, 2021
@thatmattlove thatmattlove added accepted Issue is confirmed and will be worked on bug Confirmed bug or unexpected behavior and removed possible-bug Something isn't working needs-info Issue lacks appropriate detail or requires additional information labels Dec 14, 2021
@thatmattlove
Copy link
Owner

I can confirm that this issue exists:

  1. In Firefox only
  2. On hyperglass 1.0.4
  3. Only if you use external links on both the left and right side

That being said, the issue appears to be fixed on the v2.0.0 branch, for reasons unknown. More than likely, a dependency update fixed the underlying issue accidentally. 🤷‍♂️

@thatmattlove thatmattlove added the fixed-in-next-release Issue has already been fixed in the next release label Dec 14, 2021
@pmawsonau
Copy link

Hi
Notice this issue on 1.0.4 on Firefox when all links are left. (https://lg.vocus.network)
When is v2.0.0 expected to be stable/main release?
Cheers,
Phil

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accepted Issue is confirmed and will be worked on bug Confirmed bug or unexpected behavior fixed-in-next-release Issue has already been fixed in the next release
Projects
None yet
Development

No branches or pull requests

3 participants