PROJECT#RSY00007 cmdconnector beschreven - #182
Conversation
There was a problem hiding this comment.
🟡 Changes recommended
The documentation has unresolved authentication, ZIP/import guidance, link, and example-response issues.
Get a fresh assessment by requesting another Copilot review.
Pull request overview
This PR documents AFAS Remote/CmdConnector imports in Dutch and English and adds both pages to the menu.
Changes:
- Adds Dutch and English how-to pages.
- Documents OAuth, SOAP, ZIP/base64 imports, and status checks.
- Registers both pages in the multilingual menu.
File summaries
| File | Description |
|---|---|
menustructures/multilanguage_menu.json |
Adds both how-to pages to the menu. |
markdownpages/profit/nl/howto-remotecmd.md |
Dutch documentation for AFAS Remote imports. |
markdownpages/profit/en/howto-remotecmd.md |
English documentation for AFAS Remote imports. |
Review details
Suppressed comments (10)
markdownpages/profit/en/howto-remotecmd.md:20
- 🟡 low (
severity: low) De checklist noemt geen OAuth access token, terwijl beide cURL-voorbeelden een Bearer-token in de Authorization-header vereisen. Voeg dit toe als vereiste en verwijs naar de authenticatiedocumentatie, zodat de lezer weet welke credential vóór het uitvoeren van deze voorbeelden nodig is.
- AFAS environment
- AppConnector with AFAS Remote Cmd enabled
- Command line
markdownpages/profit/en/howto-remotecmd.md:42
- 🟠 medium (
severity: medium) De tekst zegt alleen dat het importbestand base64-gecodeerd moet worden, maar de voorbeeldwaarde begint metUEsDBen is dus de base64-weergave van een ZIP-archief. Wie deze instructie letterlijk volgt en alleen de CSV encodeert, levert geen geldigattachementArchiveaan en krijgt geen werkende import. Beschrijf expliciet dat het importbestand eerst in een ZIP-archief moet worden verpakt.
The import supports different types of files, such as CSV and fixed-width files. Provide the import file as a base64-encoded file in the `<attachementArchive>` element.
markdownpages/profit/en/howto-remotecmd.md:38
- 🟡 low — Ook deze Help-link op de Engelstalige pagina verwijst naar
/help/NL/en opent daarmee de Nederlandstalige versie. Gebruik hier dezelfde/help/EN/-variant als in de rest van de Engelstalige documentatie.
For more information about the different parts of the command line, see [the Help](https://help.afas.nl/help/NL/SE/App_Cmd_Remote_Actions_Import.htm).
markdownpages/profit/en/howto-remotecmd.md:14
- 🟡 low — Deze Engelstalige pagina linkt naar
/help/NL/, waardoor lezers naar de Nederlandstalige Help-pagina worden gestuurd. Engelstalige Profit-documentatie gebruikt voor zulke links de/help/EN/-variant (bijvoorbeeldmarkdownpages/profit/en/get-connector.md:15).
See [this help article](https://help.afas.nl/help/NL/SE/App_Cmd_Remote_Actions_Import.htm) for more information about importing via the command line.
markdownpages/profit/nl/howto-remotecmd.md:20
- 🟡 low (
severity: low) De checklist noemt geen OAuth access token, terwijl beide cURL-voorbeelden een Bearer-token in de Authorization-header vereisen. Voeg dit toe als vereiste en verwijs naar de authenticatiedocumentatie, zodat de lezer weet welke credential vóór het uitvoeren van deze voorbeelden nodig is.
- AFAS Omgeving
- AppConnector met AFAS Remote Cmd geactiveerd
- Opdrachtregel (Command line)
markdownpages/profit/nl/howto-remotecmd.md:42
- 🟠 medium (
severity: medium) De tekst zegt alleen dat het importbestand base64-gecodeerd moet worden, maar de voorbeeldwaarde begint metUEsDBen is dus de base64-weergave van een ZIP-archief. Wie deze instructie letterlijk volgt en alleen de CSV encodeert, levert geen geldigattachementArchiveaan en krijgt geen werkende import. Beschrijf expliciet dat het importbestand eerst in een ZIP-archief moet worden verpakt.
De import ondersteunt verschillende soorten bestanden, zoals CSV en fixed-width bestanden. Het importbestand wordt aangeleverd als base-64 gecodeerd bestand in het `<attachementArchive>`-element.
markdownpages/profit/nl/howto-remotecmd.md:83
- 🟢 nit (
severity: nitpick) Gebruik hier het Nederlandse lidwoordde:responseis in deze documentatie een de-woord.
Om de status van een eerder uitgevoerde opdracht op te vragen, gebruik je het `GetState`-commando. Hierbij geef je het `commandId` mee dat je hebt ontvangen in het response van de `Execute`-opdracht.
markdownpages/profit/nl/howto-remotecmd.md:68
- 🟢 nit (
severity: nitpick) Gebruik hier het Nederlandse lidwoordde:responseis in deze documentatie een de-woord.
In het response zie je de `commandId` terug die je nodig hebt om de status van de opdracht op te vragen.
markdownpages/profit/nl/howto-remotecmd.md:110
- 🟢 nit (
severity: nitpick) Gebruik hier het Nederlandse lidwoordde:responseis in deze documentatie een de-woord.
In het response zie je de status van de eerder uitgevoerde opdracht terug.
markdownpages/profit/nl/howto-remotecmd.md:114
- 🟢 nit (
severity: nitpick) Gebruik hier het Nederlandse lidwoordde:responseis in deze documentatie een de-woord.
Als de import nog niet is opgepakt of nog niet is voltooid, is het element `<GetStateResult>` in het response leeg.
- Files reviewed: 3/3 changed files
- Comments generated: 1
- Review effort level: Lite
💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
There was a problem hiding this comment.
🔵 Needs a closer look
Clarify that the import file must be ZIP-packaged before Base64 encoding in both guides.
Review details
Suppressed comments (2)
markdownpages/profit/en/howto-remotecmd.md:42
- 🟠 medium — This says to Base64-encode the import file itself, but the example on line 60 starts with
UEsDB, which is a ZIP archive. If a reader encodes a raw CSV as described here, the service will not receive the archive containing/F"ImportDossier.csv"and the import will not work. State explicitly that the import file must be packaged as a ZIP archive (with the required metadata) and then Base64-encoded.
The import supports different types of files, such as CSV and fixed-width files. The import file is provided as a base64-encoded ZIP in the `<attachementArchive>` element.
markdownpages/profit/nl/howto-remotecmd.md:42
- 🟠 medium — De tekst suggereert dat je het losse importbestand alleen naar Base64 hoeft om te zetten, maar het voorbeeld op regel 60 begint met
UEsDBen is dus een ZIP-archief. Wie de CSV rechtstreeks encodeert, levert geen archive aan waarin/F"ImportDossier.csv"gevonden kan worden en krijgt geen werkende import. Beschrijf expliciet dat het importbestand als ZIP-archief (met de benodigde metadata) Base64-gecodeerd moet worden aangeleverd.
De import ondersteunt verschillende soorten bestanden, zoals CSV en fixed-width bestanden. Het importbestand wordt aangeleverd als een base64-gecodeerde ZIP in het `<attachementArchive>`-element.
- Files reviewed: 3/3 changed files
- Comments generated: 0 new
- Review effort level: Lite
There was a problem hiding this comment.
🟢 Approved
Only minor wording nits remain; no issues block approval.
Review details
Suppressed comments (4)
markdownpages/profit/nl/howto-remotecmd.md:68
- 🟢 nit — Gebruik hier “de response” in plaats van “het response”; dat is ook de vorm die in de Nederlandstalige documentatie wordt gebruikt (bijvoorbeeld
profit/nl/authentication.md:86).
In het response zie je de `commandId` terug die je nodig hebt om de status van de opdracht op te vragen.
markdownpages/profit/nl/howto-remotecmd.md:83
- 🟢 nit — Gebruik hier “de response” in plaats van “het response”; dat is ook de vorm die in de Nederlandstalige documentatie wordt gebruikt (bijvoorbeeld
profit/nl/authentication.md:86).
Om de status van een eerder uitgevoerde opdracht op te vragen, gebruik je het `GetState`-commando. Hierbij geef je het `commandId` mee dat je hebt ontvangen in het response van de `Execute`-opdracht.
markdownpages/profit/nl/howto-remotecmd.md:110
- 🟢 nit — Gebruik hier “de response” in plaats van “het response”; dat is ook de vorm die in de Nederlandstalige documentatie wordt gebruikt (bijvoorbeeld
profit/nl/authentication.md:86).
In het response zie je de status van de eerder uitgevoerde opdracht terug.
markdownpages/profit/nl/howto-remotecmd.md:114
- 🟢 nit — Gebruik hier “de response” in plaats van “het response”; dat is ook de vorm die in de Nederlandstalige documentatie wordt gebruikt (bijvoorbeeld
profit/nl/authentication.md:86).
Als de import nog niet is opgepakt of nog niet is voltooid, is het element `<GetStateResult>` in het response leeg.
- Files reviewed: 3/3 changed files
- Comments generated: 0 new
- Review effort level: Lite
No description provided.