Skip to content
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

API Design-first Article #306

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

philsturgeon
Copy link
Contributor

@philsturgeon philsturgeon commented Nov 26, 2024

@ChristopheDujarric the previous guide was good, but it conflated API Design-first and API-first design.

One of the most confusing things in the world of APIs is that these are two fundamentally different concepts...

I have focused entirely on API Design-first here, but would be happy to take a quick stab at what API-first means for the sake of SEO and telling a complete story. It could be a paragraph.

Other than that, this could definitely be better, go in deeper, have more diagrams, etc. Just wanted to fire it over as I'm on 4 hours with it already and want to make sure I get the right direction.

Copy link

netlify bot commented Nov 26, 2024

Deploy Preview for bump-content-hub ready!

Name Link
🔨 Latest commit ea67ba6
🔍 Latest deploy log https://app.netlify.com/sites/bump-content-hub/deploys/6772834f3fd33e00088262c4
😎 Deploy Preview https://deploy-preview-306--bump-content-hub.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@ChristopheDujarric
Copy link
Contributor

Thanks @philsturgeon ! It needed an update indeed.

I would love if we can still make it as "agnostic" as possible (not stick to only OpenAPI). Also, it would be great to give guidance on a good API Design-first process, such as James' ADDR.

@philsturgeon
Copy link
Contributor Author

@ChristopheDujarric lmk if this is what you had in mind. I can get stuck into ADDR more too if you like, but thats mainly just giving a name to the concepts being described here.

@ChristopheDujarric
Copy link
Contributor

@philsturgeon thank you for the update, this looks great! Can you please check about the failing test?

@philsturgeon
Copy link
Contributor Author

@ChristopheDujarric fixed!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants