Skip to content

[OSDEV-1698] Fix link for submit another location button#505

Merged
VadimKovalenkoSNF merged 12 commits intomainfrom
OSDEV-1698-fix-submit-another-location-link
Feb 13, 2025
Merged

[OSDEV-1698] Fix link for submit another location button#505
VadimKovalenkoSNF merged 12 commits intomainfrom
OSDEV-1698-fix-submit-another-location-link

Conversation

@VadimKovalenkoSNF
Copy link
Contributor

@VadimKovalenkoSNF VadimKovalenkoSNF commented Feb 10, 2025 •
edited by atlassian bot
Loading

Fix OSDEV-1698
Refactor link of Submit another Location button to point to the search by name and address form.

@VadimKovalenkoSNF VadimKovalenkoSNF self-assigned this Feb 10, 2025
@barecheck
Copy link

barecheck bot commented Feb 10, 2025 •
edited
Loading

Dedupe Hub App | Unittest test suite - Code coverage report

Total: 56.14%

Your code coverage diff: 0.00% ▴

✅ All code changes are covered

@barecheck
Copy link

barecheck bot commented Feb 10, 2025 •
edited
Loading

Countries App | Unittest test suite - Code coverage report

Total: 100%

Your code coverage diff: 0.00% ▴

✅ All code changes are covered

@barecheck
Copy link

barecheck bot commented Feb 10, 2025 •
edited
Loading

Contricleaner App | Unittest test suite - Code coverage report

Total: 98.91%

Your code coverage diff: 0.00% ▴

✅ All code changes are covered

roman-stolar
roman-stolar previously approved these changes Feb 12, 2025
View reviewed changes
Copy link
Contributor

@roman-stolar roman-stolar left a comment

Choose a reason for hiding this comment

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

LGTM

roman-stolar
roman-stolar reviewed Feb 12, 2025
View reviewed changes
mazursasha1990
mazursasha1990 previously approved these changes Feb 12, 2025
View reviewed changes
Copy link
Contributor

@mazursasha1990 mazursasha1990 left a comment

Choose a reason for hiding this comment

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

LGTM

mazursasha1990
mazursasha1990 previously approved these changes Feb 13, 2025
View reviewed changes
Copy link
Contributor

@mazursasha1990 mazursasha1990 left a comment

Choose a reason for hiding this comment

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

LGTM

@roman-stolar
Copy link
Contributor

roman-stolar commented Feb 13, 2025

Please check FE tests.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 13, 2025

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 SonarQube Cloud

mazursasha1990
mazursasha1990 approved these changes Feb 13, 2025
View reviewed changes
Copy link
Contributor

@mazursasha1990 mazursasha1990 left a comment

Choose a reason for hiding this comment

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

LGTM

@coderabbitai coderabbitai bot mentioned this pull request Feb 21, 2025
Merged
This was referenced Mar 6, 2025
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vladsha-dev vladsha-dev Awaiting requested review from vladsha-dev

@roninzp roninzp Awaiting requested review from roninzp

2 more reviewers

@mazursasha1990 mazursasha1990 mazursasha1990 approved these changes

@roman-stolar roman-stolar roman-stolar approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@VadimKovalenkoSNF VadimKovalenkoSNF

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@VadimKovalenkoSNF @roman-stolar @mazursasha1990