MingweiSamuel / lcu-schema

Script to serve LCU spec site
http://www.mingweisamuel.com/lcu-schema/
MIT License
25 stars 3 forks source link

Fix: remove `ConvertFrom-Json` for `/Help` #6

Closed mikaeldui closed 2 years ago

mikaeldui commented 2 years ago

Seems PowerShell had parsed it already.

The action was complaining about an illegal @ at the start, and it was probably caused by the string representation of the PowerShell object.

image

mikaeldui commented 2 years ago

I wonder if the Epytext and Console formats could be useful for anyone, in addition to the Full format?

Would it be more correct to have files like this:

The Brief format might not be too useful? Though I did find its compact format somewhat useful for the events.

And, what about converting /Help to the Open API and Swagger formats?

MingweiSamuel commented 2 years ago

Sounds good to me to have all four, doesn't hurt, if you want to make a PR. Otherwise Ill find a time to do it later.

Also I guess the

# /Help is missing the `Content-Type: application/json` header when logged-in.

Can be removed too

mikaeldui commented 2 years ago

Turns out that only Console has the REST related things, like explicit HTTP method and URL.