Update OAuth Authorization Code Flow documentation - #181
Conversation
…er order and add refresh token retrieval steps
There was a problem hiding this comment.
🟡 Changes recommended
The response fields list is currently malformed and documents an incorrect token_type value, which can mislead consumers.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
review nodig: ja
labels: 📚 leerzaam — deze PR is leerzaam omdat hij expliciet laat zien hoe je met een refresh token een nieuw access/refresh token ophaalt binnen de Authorization Code Flow met PKCE.
Deze PR actualiseert de documentatie voor de OAuth Authorization Code Flow (met PKCE) voor Profit door de parameterlijst/voorbeeld-URL te corrigeren en een expliciete refresh-token stap toe te voegen.
Changes:
- Scope verwijderd uit de autorisatie-request parameters en het cURL URL-voorbeeld (NL/EN).
- Stap toegevoegd voor het ophalen van een nieuw access token (en refresh token) via
grant_type=refresh_token(NL/EN). - Stapnummering aangepast om de refresh-token stap op te nemen (NL/EN).
File summaries
| File | Description |
|---|---|
| markdownpages/profit/nl/authentication.md | Past Authorization Code Flow-parameters en voegt refresh-token stap toe in NL-documentatie. |
| markdownpages/profit/en/authentication.md | Past Authorization Code Flow-parameters en voegt refresh-token stap toe in EN-documentatie. |
Review details
- Files reviewed: 2/2 changed files
- Comments generated: 2
- Review effort level: Lite
💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
🔵 Needs a closer look
De response-veldenlijst voor de authorization code flow heeft dubbele/misgenummerde output door een gecombineerde token_type/expires_in regel, wat in de gerenderde documentatie fout gaat.
Review details
Suppressed comments (2)
markdownpages/profit/nl/authentication.md:170
- 🟠 medium: Deze lijst met response-velden combineert token_type en expires_in in één item via een
<br>, terwijlexpires_inook nog als apart item eronder staat. Dit geeft dubbele/misgenummerde output in de gerenderde documentatie; maak er twee aparte lijstitems van.
3. token_type: Bearer
markdownpages/profit/en/authentication.md:171
- 🟠 medium: This response-fields list item combines
token_typeandexpires_inusing a<br>, whileexpires_inis also listed again on the next line. This produces duplicated/misnumbered output in rendered docs; keeptoken_typeandexpires_inas separate list items.
3. token_type: Bearer
- Files reviewed: 2/2 changed files
- Comments generated: 0 new
- Review effort level: Lite
Update OAuth Authorization Code Flow documentation to correct parameter order and add refresh token retrieval steps