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

Links to github in Units section #5260

Closed
13DaGGeR opened this issue Jul 22, 2023 · 10 comments
Closed

Links to github in Units section #5260

13DaGGeR opened this issue Jul 22, 2023 · 10 comments
Labels
status: new issue Is added to every issue to indicate that it has not been reviewed yet

Comments

@13DaGGeR
Copy link

Describe the bug
The unit section of the lobby application has links to users (?). Clicking on such a link will result in a GitHub page displayed in the lobby app, breaking the units section interface.

To Reproduce

  1. Got to Units tab
  2. Click on UEF t1 engineer (or any other unit, as far as I can see)
  3. Click the "Compare" yellow button
  4. Click the "UEL0105" link under the unit title
  5. See "Error Looks like something went wrong!" GitHub page

Expected behavior
Either:

  1. replace the links with simple text
  2. or change the URL to a working one and allow a user to close this page afterward somehow.

Screenshots
image
image

Additional context
FAF version 2023.6.1, launched on win 11.

@13DaGGeR 13DaGGeR added the status: new issue Is added to every issue to indicate that it has not been reviewed yet label Jul 22, 2023
@Garanas
Copy link
Member

Garanas commented Jul 22, 2023

I don't think this is related to the game, but to the tool behind the units tab. @Sheikah45 do you know what that tool is?

@Sheikah45
Copy link
Member

Depends which unit database they are using

@Garanas
Copy link
Member

Garanas commented Jul 22, 2023

This is the one that is in the client

@Sheikah45
Copy link
Member

There are two in the client depending on which one people choose in the settings

@MostLostNoob
Copy link
Contributor

The UnitIDs in both the Rackover & Spooky DB in the client will lead to the GitHub pages.

At the present time, links are indeed returning with Error messages, so that needs to be fixed.

@13DaGGeR, regarding getting back to the unit selection page, you just need to right-click the GitHub page & select Go Back, so there's nothing that needs to be fixed with regards to that. With that said, there's no mention of that to be seen, so adding a header or something that is easily seen may be worth looking into.

@Garanas
Copy link
Member

Garanas commented Jul 26, 2023

What are the repositories of those applications?

@MostLostNoob
Copy link
Contributor

MostLostNoob commented Jul 26, 2023

If I'm not mistaken, these are the repos for the databases:

Spooky: https://github.com/FAForever/spooky-db

Rackover: https://github.com/FAForever/UnitDB

Here's the websites for them:

Spooky: https://faforever.github.io/spooky-db/#/

Rackover: https://unitdb.faforever.com/

To expand on my previous post, following the links from the UnitID links on the webpages results in those bp files being loaded from GitHub without issues; however, doing the same from within the client results in a GitHub error page being loaded.

@Garanas
Copy link
Member

Garanas commented Jul 26, 2023

I don't know what more actions are required to get the changes into production. Do you know that @Sheikah45 ?

@MrRowey
Copy link
Member

MrRowey commented Jul 26, 2023

This is a Client issue not the UnitDBs as you can use the web version and the link works fine

@MrRowey
Copy link
Member

MrRowey commented Nov 17, 2023

Closing as the link are working

@MrRowey MrRowey closed this as not planned Won't fix, can't repro, duplicate, stale Nov 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: new issue Is added to every issue to indicate that it has not been reviewed yet
Projects
None yet
Development

No branches or pull requests

5 participants