/corporations/{corporation_id}/customs_offices/ doesn't return Customs Offices within Planetary Skyhooks

[weeeeems]

Feature Request

When requesting Corporation Customs Offices only Orbital Customs Offices are shown, and not the Customs Offices within Orbital Skyhooks. They should be included either here (makes most sense) or on the Structures endpoint.

Use case

Corporation and alliance leaders, third party apps, the unpaid IT nerds critical to the running of any large group.

We use first and third party tools to monitor our structures and ensure things like access and taxes are aligned. This is not easily done even in the ingame client (hundreds of clicks). This is already available for existing Customs Offices so seems like this was just missed when adding Orbital Skyhooks.

Authentication

Does this feature require a new scope, an existing scope, or no scope? E.g.:

Existing scope esi-planets.read_customs_offices.v1

Example return

Propose what an example return might look like. E.g.:

Should make no difference if its an Orbital Skyhooks or Customs Office. The only difference would be a wider reinforce_exit_end and reinforce_exit_start which would easily identify it as a Skyhook.

[
  {
    "alliance_tax_rate": 0.019999999552965164,
    "allow_access_with_standings": true,
    "allow_alliance_access": true,
    "bad_standing_tax_rate": 0.10000000149011612,
    "corporation_tax_rate": 0.019999999552965164,
    "excellent_standing_tax_rate": 0.019999999552965164,
    "good_standing_tax_rate": 0.019999999552965164,
    "neutral_standing_tax_rate": 0.10000000149011612,
    "office_id": {office_id},
    "reinforce_exit_end": 14,
    "reinforce_exit_start": 10,
    "standing_level": "terrible",
    "system_id": {system_id},
    "terrible_standing_tax_rate": 0.10000000149011612
  },
  ...
]

Checklist

Check all boxes that apply to this issue:

• [X] Feature request description is provided
• [X] Use case exists
• [ ] Feature requires a new route
• [X] Feature adds data to existing route
• [ ] Feature requires new auth scope
• [X] Feature can reuse existing scope
• [ ] Feature does not require auth
• [ ] Meta feature, applies to all routes

[soratidus999]

Thankyou for documenting this, this was raised with CCP and CSM day 4-5 of equinox. But i did not have skyhooks or pocos to show example returns for the data