Skip to content

fix: correct SERVICE_TASK_START comment, 2 MB thresholds in docs, bump to 1.0.1 - #4

Merged
Klaas-Ritense merged 1 commit into
valtimo-platform:mainfrom
mazix-DA:fix/correct-comment-and-docs
Jun 4, 2026
Merged

fix: correct SERVICE_TASK_START comment, 2 MB thresholds in docs, bump to 1.0.1#4
Klaas-Ritense merged 1 commit into
valtimo-platform:mainfrom
mazix-DA:fix/correct-comment-and-docs

Conversation

@mazix-DA

@mazix-DA mazix-DA commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

Wijzigingen (Deze PR bevat uitsluitend documentatie- en commentaarcorrecties. Er is geen functionele code gewijzigd.)

GraphMailPlugin.kt — NOTE-commentaarblok boven @PluginAction

  • USER_TASK_CREATE gecorrigeerd naar SERVICE_TASK_START (comment verwees naar verkeerd activity type)
  • Suggestie voor procesvariabele als idempotency-token verwijderd
  • Vervangen door beschrijving van het restrisico: at-least-once delivery,
    geen idempotency-guard, en dat een procesvariabele geen betrouwbare guard
    is omdat die transactioneel is en bij een retry mee terugrolt

documentation/plugin.md — documentatiecorrecties

  • Alle vier "3 MB"-verwijzingen gecorrigeerd naar "2 MB" (conform INLINE_ATTACHMENT_THRESHOLD_BYTES = 2 MiB in de code)
  • Upload-sessiedrempel verduidelijkt: slaat aan zodra één losse bijlage óf
    het totaal boven 2 MB komt (op drie plekken)
  • Beheerdersconsent-notitie toegevoegd: Mail.Send en Mail.ReadWrite zijn
    applicatiemachtigingen die admin consent vereisen
  • Queue-size-kanttekening toegevoegd aan de pool-config-sectie

Versie

  • Gebumpt van 1.0.0 naar 1.0.1 (plugin.properties ×2, package.json, getting-started.md)

@Klaas-Ritense
Klaas-Ritense merged commit 1011dc9 into valtimo-platform:main Jun 4, 2026
2 checks passed
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