Skip to content

PROJECT#RSY00007 cmdconnector beschreven - #182

Merged
Eric Zwaal (ezw2000) merged 4 commits into
mainfrom
cmdconnector-beschreven
Sep 14, 2026
Merged

Eric Zwaal (ezw2000) merged 4 commits into
mainfrom
cmdconnector-beschreven

Conversation

@ezw2000

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI lite review requested due to automatic review settings September 14, 2026 17:41
@ezw2000
Eric Zwaal (ezw2000) enabled auto-merge (squash) September 14, 2026 17:41

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 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 met UEsDB en is dus de base64-weergave van een ZIP-archief. Wie deze instructie letterlijk volgt en alleen de CSV encodeert, levert geen geldig attachementArchive aan 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 (bijvoorbeeld markdownpages/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 met UEsDB en is dus de base64-weergave van een ZIP-archief. Wie deze instructie letterlijk volgt en alleen de CSV encodeert, levert geen geldig attachementArchive aan 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 lidwoord de: response is 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 lidwoord de: response is 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 lidwoord de: response is 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 lidwoord de: response is 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.

Comment thread markdownpages/profit/en/howto-remotecmd.md Outdated

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔵 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 UEsDB en 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

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 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

@ezw2000
Eric Zwaal (ezw2000) merged commit da51a6c into main Sep 14, 2026
10 checks passed
@ezw2000
Eric Zwaal (ezw2000) deleted the cmdconnector-beschreven branch September 14, 2026 18:11
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