Skip to content

fix(new): update service port and server address to 4027 in configuration files#1165

Merged
guimoreirar merged 1 commit intomainfrom
hotfix/bank-transfer-port
Apr 2, 2026
Merged

fix(new): update service port and server address to 4027 in configuration files#1165
guimoreirar merged 1 commit intomainfrom
hotfix/bank-transfer-port

Conversation

@guimoreirar
Copy link
Copy Markdown
Member

@guimoreirar guimoreirar commented Apr 2, 2026

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 2, 2026
@guimoreirar guimoreirar requested a review from a team as a code owner April 2, 2026 14:40
@guimoreirar guimoreirar changed the title fix(plugin-br-bank-transfer): update service port and server address to 4027 in configuration files fix(new): update service port and server address to 4027 in configuration files Apr 2, 2026
Copy link
Copy Markdown
Contributor

@gandalf-at-lerian gandalf-at-lerian left a comment

Choose a reason for hiding this comment

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

Clean and consistent port change from 8080 → 4027 across all config surfaces (deployment, configmap, values, values-template, README). No mismatches. LGTM.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 2, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 440b820c-046f-4e94-8f7d-073a0bf39669

📥 Commits

Reviewing files that changed from the base of the PR and between f045ca2 and dd4e2a0.

📒 Files selected for processing (5)
  • charts/plugin-br-bank-transfer/README.md
  • charts/plugin-br-bank-transfer/templates/configmap.yaml
  • charts/plugin-br-bank-transfer/templates/deployment.yaml
  • charts/plugin-br-bank-transfer/values-template.yaml
  • charts/plugin-br-bank-transfer/values.yaml

Walkthrough

Port configuration update across the Brazilian Bank Transfer Helm chart, changing the default exposed service port and corresponding server address from 8080 to 4027 in documentation, templates, and configuration files.

Changes

Cohort / File(s) Summary
Documentation and Configuration
charts/plugin-br-bank-transfer/README.md, charts/plugin-br-bank-transfer/values.yaml, charts/plugin-br-bank-transfer/values-template.yaml
Updated default port values from 8080 to 4027 for bankTransfer.service.port and bankTransfer.configmap.SERVER_ADDRESS across documentation and Helm values files.
Kubernetes Resource Templates
charts/plugin-br-bank-transfer/templates/deployment.yaml, charts/plugin-br-bank-transfer/templates/configmap.yaml
Modified container port exposure and ConfigMap environment variable from 8080 to 4027 in deployment and configuration templates.

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

@guimoreirar guimoreirar merged commit 0cf556d into main Apr 2, 2026
3 of 6 checks passed
@guimoreirar guimoreirar deleted the hotfix/bank-transfer-port branch April 2, 2026 14:53
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.

3 participants