Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Register
  • Sign in
  • erp5 erp5
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Merge requests 142
    • Merge requests 142
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Jobs
  • Commits
Collapse sidebar
  • nexedinexedi
  • erp5erp5
  • Merge requests
  • !1393

WIP: base: Configure substitution's "ignore missing" on notification messages

  • Review changes

  • Download
  • Patches
  • Plain diff
Open Jérome Perrin requested to merge feat/notification-message-ignore-missing into master Apr 12, 2021
  • Overview 3
  • Commits 1
  • Pipelines 1
  • Changes 6

TextDocument substitution API already supported an ignore_missing argument, so that we can programmatically control whether missing entries should be rendered as ${variable} or should raise an error. This makes sense for some "important" notifications, where not sending a message would be less problematic than sending a message where some variables have not been substituted.

This extends this concept by allowing to configure as a property on the notification message whether missing entries should be ignored. This default to "ignore" to maintain compatibility.

Edited Apr 19, 2021 by Jérome Perrin
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: feat/notification-message-ignore-missing
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7