-
Notifications
You must be signed in to change notification settings - Fork 409
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
Vanity ship importing needs fixing #67
Comments
I would not assume it is the same ship. There were cases when ship stats differ (after t1 hull rebalance, vanity variation wasn't updated, e.g. ishukone watch scorpion). Thus hybrid approach won't be a very good idea. |
Point noted, that would require us to trust CCP to update accordingly. -_- A custom group would seem to be the best course of action, then, unless there is objection. I've got a local branch with these changes - currently with all vanity ships noted in the market service. However, not all vanity ships are part of the new, replenish able skinned ships. Should we just include those that were added with Rubicon 1.3? |
It might be good solution, but short-term: i think chances that ccp will change painting approach rather sooner than later (to single typeID with multiple paint 'modules' or whatever). When they do it, we might need to do database conversion jobs which are always pain in the ass - both to prooftest and maintain in future. Considering this, i would prefer 'mapping' to original typeIDs, even though it might be not very accurate in some edge cases (CCP probably even fixed ishukone scorp, i didn't check). This could also be used to maintain map of old names to typeIDs if we will want to support old module names for importing in future. |
Totally forgot about this issue. I think we need to do this at EOS level, so that way we can easily do the conversion from We could also import the mapping into the market service so that we can force publish settings. I'll take another look at this soon. |
Actually, this was fairly easy: #97 Wasn't really sure where to put the conversion map, so I made a new file for it. And I'm also bad at naming files, so feel free to rename it to something better =3 |
Just for the sake of documentation: I suggest to move it to service layer, or at least move from DB-related eos part to fit importing part. |
Importing of vanity ships is somewhat broken. The import works, but there's no way of accessing it unless you search for the name as vanity ships are forcefully excluded from Pyfa's market.
This was not really an issue when they were limited ships, but now with the new skinning feature they are more common. Case in point: Police Pursuit Comet. Although these have the same stats as their regular counterpart, those who find a kill on the killboard may want to import the kill, and will not know to edit the kill to show the regular ship.
I propose one of three things:
skinned: unskinned
relationship in the market service. On import, simply compare and save as the unskinned ship. However, this has an issue in that the price will not be accurate, and those who import a Police Pursuit Comet will not expect it to show up in the regular comet fittings.I am not sure if CCP maintains the relationship internally. I don't think so, as there is no relationship in the variations tab in-game. If there is no internal relationship, it might be a pain to keep up to date when new skins are released.
The text was updated successfully, but these errors were encountered: