Skip to content
Snippets Groups Projects

this changes prevents trying to replace placeholders not used in the file....

Merged Carlos Vega requested to merge fix/515 into master

this changes prevents trying to replace placeholders not used in the file. However, if a field value is None it will still fail. We need to discuss the approach to follow if a required field is none, should it be replaced by the text None or how can we propagate an error to the frontend?

Merge request reports

Pipeline #70383 passed

Pipeline passed for 24f5668f on fix/515

Approval is optional

Merged by Piotr GawronPiotr Gawron 1 year ago (Mar 27, 2023 10:23am UTC)

Merge details

  • Changes merged into master with bd6c9d30.
  • Deleted the source branch.

Pipeline #70474 passed

Pipeline passed for bd6c9d30 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading