Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat(condo): DOMA-10366 name for bank account #5331

Merged
merged 5 commits into from
Oct 9, 2024
Merged

feat(condo): DOMA-10366 name for bank account #5331

merged 5 commits into from
Oct 9, 2024

Conversation

YEgorLu
Copy link
Contributor

@YEgorLu YEgorLu commented Oct 7, 2024

After payment split we will have to show managing company that money went on multiple bank accounts, so adding name for them, instead of showing all with same managing company name

@YEgorLu YEgorLu added ✋🙂 Review please Comments are resolved, take a look, please 🚨 Migrations We have a database migrations here! labels Oct 7, 2024
@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Oct 8, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@YEgorLu YEgorLu merged commit 8c280fc into main Oct 9, 2024
26 checks passed
@YEgorLu YEgorLu deleted the feat/condo/DOMA-10366/name-for-bank-account branch October 9, 2024 05:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dkoviazin dkoviazin dkoviazin approved these changes

@abshnko abshnko abshnko approved these changes

Assignees
No one assigned
Labels
🚨 Migrations We have a database migrations here! ✋🙂 Review please Comments are resolved, take a look, please
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@YEgorLu @dkoviazin @abshnko