-
Notifications
You must be signed in to change notification settings - Fork 40
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Parser ignoring responses #9
Comments
I have the same problem, I take material from the documentation, but any responses other than 200 are not displayed |
@RodionovSlsUp I just managed to fix some of it, you'll now be able to render multiple responses: sheinfeld/idoc My next step will be to render the response data parameters like the body ones: https://i.imgur.com/BGQRj3K.png Stay tuned |
Also just updated the core JS library to it's latest version, looks way better! |
@sheinfeld in fact, I did about the same thing myself :) There are very few response fields, it's true |
@sheinfeld Thank you very much, you are cool |
Hey @RodionovSlsUpx
Unfortunately we can’t actually used forked ones, but I’ll make an
independent version so we and others can make use of an updated version!
I’ll try have it ready by the end of the week!
On Tue, 30 Mar 2021 at 21:14, RodionovSlsUp ***@***.***> wrote:
@sheinfeld <https://github.com/sheinfeld>
Thomas, tell me, is it possible to use your fork through the composer?
And also, is there an understanding, in terms of time, when it will be
possible to make the response fields?
Thank you very much, you are cool
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#9 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AIGTXAFSRGKCFCNNOWNTC23TGIWJ5ANCNFSM4S3EXH4Q>
.
--
Best regards,
Tomas C. Lopes
(Sent from my mobile)
|
@sheinfeld |
@sheinfeld by the way, your 500 error is marked as a success. I have not figured out how to elegantly solve this problem and moved the blanks with an error to the config. And I draw the name according to the code from the config.You can see examples in the screenshot |
@RodionovSlsUp I'm looking at it now :D |
@ovac will this be merged back into your repo? Thanks for the hard work btw.. idoc is appreciated very much over here! |
BTW, this issue is blocking the phpunit tests driven by hotmeteor/spectator.. if that would work its a great combination. |
@ronaldwanink, Thank you very much for your kind comment. I will be merging the feature with the next release. I however will want all tests to run successfully though. |
Good afternoon, first of all let me thank you for this amazing package!
I am trying to add the responses to the documentation, but once generated I only get the default 200 and nothing else.
/** * @response 200 { * "status": "success", * "message": "Your request was successfully created" * } * * @response 500 { * "status": "error", * "message": ["Custom error message"] * } */
Do I need to add/change anything?
The text was updated successfully, but these errors were encountered: