-
Notifications
You must be signed in to change notification settings - Fork 90
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
create process with netlify's dev team to add new attributes and calls <EOM> #130
Comments
@fool, has this process been solved? |
nope. Or at least, if it has, nobody has mentioned to me that it is now a step in the "releasing/changing API endpoints" process and I'd bet it wasn't added there (if there is even a there, there). It's probably worth taking up as adding to to the project template we're using now in notion though. |
Yes I agree. It seems to me the OpenAPI definition has few discrepancies. Having a process for updating it might be a good way to ensure our clients and the API documentation are in sync with the API. |
Yes! As we keep moving towards more "docs-driven devlopment", we're hoping to have it be part of the API dev process to PR changes to the OpenAPI definition before working on changes. (Of course, we'll also need some sort of process for finding and correcting existing gaps.) |
@verythorough - great to hear!
The process you mention is "just" someone making time for this:
#131
which I realize doesn't say anything new but it's effectively "audit the
code"
…On Fri, Sep 20, 2019 at 1:35 PM Jessica Parsons ***@***.***> wrote:
Yes! As we keep moving towards more "docs-driven devlopment", we're hoping
to have it be part of the API dev process to PR changes to the OpenAPI
definition *before* working on changes.
(Of course, we'll also need some sort of process for finding and
correcting existing gaps.)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#130?email_source=notifications&email_token=AABX5IGV67OMP7C52GHO2QLQKUXXVA5CNFSM4GPBHBNKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7HZ3QQ#issuecomment-533700034>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AABX5IAOUPQNRIE4GBTGVTLQKUXXVANCNFSM4GPBHBNA>
.
|
Yeah, I suppose "process" over-sells it a bit. 😛 |
No description provided.
The text was updated successfully, but these errors were encountered: