mtgjson / mtgjson-website

MTGJSON Documentation Front-End Application built with VitePress
https://mtgjson.com
MIT License
36 stars 24 forks source link

Typos, undocumented fields and incorrectly documented fields. #595

Closed JochemKuijpers closed 11 months ago

JochemKuijpers commented 1 year ago

Location of update needed

On several model pages. I'll list them per item.

What does the documentation look like?

See below.

What should the documentation look like?

By data model page:

ForeignData

Booster

Legalities

Set

Card (Set)

Card (Token)

Additional context

I've posted about these findings in the #documentation Discord channel as well. Feel free to reach out to me there or here for more information if necessary.

JochemKuijpers commented 1 year ago

I've updated the issue and removed this:

PurchaseUrls

  • A field is missing from the documentation but seen in the AllPrintings data: cardKingdom.

This actually wasn't an issue, I just missed the first line in the TypeScript signature when copying the data.

ZeldaZach commented 1 year ago

cc @staghouse

staghouse commented 1 year ago

I'm not going to have time soon to fix these issues. Any help is appreciated.