/characters/{character_id}/notifications/ - unknown notification type (6040)

[soratidus999]

Bug

Regression of #1368

Some notifications returned from /characters/{character_id}/notifications/ are not returning their type, instead they are returning an "unknown notification type (6040)" or "unknown notification type (6041)" breaking validation of this response data. Interfering with the normal usage of this endpoint.

This is occuring because the respective notification types have not been added to eve-glue

esi/eve-glue#35 has been merged to fix 280-285

Request

GET http://characters/character_id/notifications/?datasource=tranquility

Response

Status Code

200

Headers

I didnt record headers this time around

Body

...
  {
    "notification_id": 1234567890,
    "sender_id": 1234567890,
    "sender_type": "character",
    "text": "corpID: 1234567890\n",
    "timestamp": "2024-06-17Txx:yy:00Z",
    "type": "unknown notification type (6040)"
  },
...

Expected

200

...
  {
    "notification_id": 1234567890,
    "sender_id": 1234567890,
    "sender_type": "character",
    "text": "corpID: 1234567890\n",
    "timestamp": "2024-06-17Txx:yy:00Z",
    "type": "StructureLowReagentsAlert"
  },
...

Checklist

Check all boxes that apply to this issue:

• Bug description is provided
• Request path is provided
• Response status code is provided
• Response headers are provided
• Response body is provided
• Expected response is provided

[soratidus999]

unknown notification type (283) introduced by PATCH NOTES FOR 2024-07-04.1

[soratidus999]

unknown notification type (6040) Is being passed to ESI, not sure if this is new but we've only just noticed it

I thought these were supposed to be be persistent notifications, so i have no followed up these on eve-glue

[soratidus999]

/v6/characters/{character_id}/notifications/ Addresses most of these notifications, 6040 is still outstanding

[soratidus999]

Updated post to reflect 6040 StructureLowReagentsAlert and 6041 StructureNoReagentsAlert

[steven-noorbergen]

This should be fixed now.