Skip to content

fix(new): update DEFAULT_TENANT_ID to allow empty default value in configmap#1182

Merged
guimoreirar merged 1 commit intomainfrom
hotfix/bank-transfer-default-tenant
Apr 10, 2026
Merged

fix(new): update DEFAULT_TENANT_ID to allow empty default value in configmap#1182
guimoreirar merged 1 commit intomainfrom
hotfix/bank-transfer-default-tenant

Conversation

@guimoreirar
Copy link
Copy Markdown
Member

Midaz Pull Request Checklist

Pull Request Type

  • Midaz
  • Plugin Access Manager
  • Plugin CRM
  • Reporter
  • Plugin Fees
  • Plugin BR PIX Direct JD
  • Plugin BR PIX Indirect BTG
  • Otel Collector
  • Pipeline
  • Documentation
  • Fetcher
  • Matcher
  • Flowker
  • Underwriter

Checklist

Please check each item after it's completed.

  • I have tested these changes locally.
  • I have updated the documentation accordingly.
  • I have added necessary comments to the code, especially in complex areas.
  • I have ensured that my changes adhere to the project's coding standards.
  • I have checked for any potential security issues.
  • I have ensured that all tests pass.
  • I have updated the version appropriately (if applicable).
  • I have confirmed this code is ready for review.

Additional Notes

Obs: Please, always remember to target your PR to develop branch instead of main.

@guimoreirar guimoreirar self-assigned this Apr 10, 2026
@guimoreirar guimoreirar requested a review from a team as a code owner April 10, 2026 19:05
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 10, 2026

Caution

Review failed

Pull request was closed or merged during review

Walkthrough

Modified a ConfigMap template for the plugin-br-bank-transfer Helm chart by changing the default value of the DEFAULT_TENANT_ID environment variable from a UUID string to an empty string. This is a configuration-level adjustment with no changes to conditional logic or other environment keys.

Changes

Cohort / File(s) Summary
ConfigMap Template
charts/plugin-br-bank-transfer/templates/configmap.yaml
Changed DEFAULT_TENANT_ID Helm default value from 11111111-1111-1111-1111-111111111111 to an empty string in the Multi-Tenancy section.

Comment @coderabbitai help to get the list of available commands and usage tips.

@guimoreirar guimoreirar changed the title fix(bank-transfer): update DEFAULT_TENANT_ID to allow empty default value in configmap fix(new): update DEFAULT_TENANT_ID to allow empty default value in configmap Apr 10, 2026
@guimoreirar guimoreirar merged commit 818a089 into main Apr 10, 2026
2 of 4 checks passed
@guimoreirar guimoreirar deleted the hotfix/bank-transfer-default-tenant branch April 10, 2026 19:06
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