-
Notifications
You must be signed in to change notification settings - Fork 89
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 new release with recent changes #243
Comments
+1 A release would be great, right now my code is breaking because hasn't been incorporated into a release yet. Also, the |
I am having the exact same issue @DougAnderson444 I am glad I am not the only one. One thing that I have been attempting is to do is make a Crate based off of the OpenAPI schemas posted by Cloudflare in this repository: https://github.com/cloudflare/api-schemas I think this should be possible but I have not found any tooling that will work with APIs this large/complex, they all have one little thing that is not supported. If you have any ideas I would love to hear them and maybe we can get a working solution up and running. If done properly, an automated pipeline should be able to handle most of this work (in theory). Thanks for the response |
Is there any reason this has been held up? It would really benefit my (and others) work for this to be done. I am more than willing to aid in this process in any way I can, please just let me know what needs to be done. Thanks! Tagging @hrushikeshdeshpande as he was the last one I have seen active on this repository. |
I’m not part of Cloudflare, but I was experiencing the exact same issue. Regarding your idea, I’ve seen something similar implemented in the async-stripe library. It works great for me and essentially follows the approach you’re suggesting, just tailored for Stripe. For now, I only need the DNS endpoints, but if Cloudflare doesn’t address this soon, it might be worth launching a community version with improvements and fixes. I also find the existing documentation somewhat lacking. |
The changes merged in this PR: #241 fixed an issue with the API that makes an endpoint (
ListDnsRecords
) unusable. The code has been reviewed and merged, a new release should be generated.Keeping these changes unreleased for weeks/months will generate more issues and general help requests as people experience broken behavior and (understandably) don't check the commit log to see if their problem has been resolved.
Some other low hanging fruit that could be tossed into this Crate release:
repository
fieldrepository
field #235Eq, PartialEq
forDnsRecord
Eq
forDnsContent
#226Eq
forDnsContent
#227Thank you for your time, I would love to see this repository to get some TLC and bring it up to date for the latest Cloudflare API. It would be very useful for my application along with many others.
The text was updated successfully, but these errors were encountered: