Suggestion: Add TC numbers to the technique list about temtem-api HOT 2 CLOSED

Hey again, thanks for the suggestion!

I opted to do two things:

1. Add the /api/training-courses endpoint in 2939b52. The data under it looks something like this:

[{
    "number": "TC001",
    "technique": "Tsunami",
    "type": "Water",
    "location": "Complete Gone with the Sillaro. Requires the Surfboard.",
    "locationType": "quest"
  },
  {
    "number": "TC002",
    "technique": "Stone Wall",
    "type": "Earth",
    "location": "Windward Fort prison floor, cell in the water. Requires the Surfboard.",
    "locationType": "found"
  }]

2. In fe63bb4 I made it so the /api/temtems endpoint can take a expand query parameter for a number of fields in the past, including techniques. Now, if you do /api/temtems?expand=techniques, the API will fill out all the technique data, including adding training course information on a trainingCourse key with an item in the same shape as what you'd get a list of from /api/training-courses.

I decided to do it this way, as the dictionary approach, while good for looking up, isn't common convention for an API serving JSON, and makes it more difficult if for example someone wanted to just display a list of all available training courses. But by doing the endpoint as a list, and allowing it to be added by the API where most people would want it, hopefully it achieves what we want.

What do you think?

from temtem-api.

Looks good to me!

from temtem-api.