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

[Prism] Asset viewer locations. Player owned structures #51

Closed
Slivo-fr opened this issue Aug 4, 2017 · 0 comments
Closed

[Prism] Asset viewer locations. Player owned structures #51

Slivo-fr opened this issue Aug 4, 2017 · 0 comments
Assignees
Labels
bug

Comments

@Slivo-fr
Copy link
Collaborator

@Slivo-fr Slivo-fr commented Aug 4, 2017

From following forum bug report : http://evehq.co/forum/viewtopic.php?f=5&t=456

Prism asset viewer display location that are not station with an ID, including the 'head' of the pilot with implants

Steps to reproduce

  1. Add a character with implants and some stuff in a citadel to EveHQ
  2. Open Prism asset viewer
  3. Pull data for this character

Bug

  1. A location with the pilot ID is shown with trained skills and actual clone implants
  2. Citadel displayed name are their ID

Expected

Only real location should be displayed
Citadel should be displayed with their ingame name

Notes

This part of EveHQ has been changed by paulkpeterson that added the flat parameter to the API call so we can retrieve citadel assets. It's still in a sort of early state and need to be worked on.

Fixed in

2.31.6

@Slivo-fr Slivo-fr added the bug label Aug 4, 2017
@Eshva Eshva self-assigned this Aug 10, 2017
@Eshva Eshva added this to In Progress in EveHQ Legacy Aug 11, 2017
Eshva added a commit that referenced this issue Aug 13, 2017
@Eshva Eshva moved this from In Progress to Done in EveHQ Legacy Aug 13, 2017
@Eshva Eshva changed the title [Prism] Asset viewer locations [Prism] Asset viewer locations. Player owned structures Aug 20, 2017
@Eshva Eshva closed this Aug 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.