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.

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

Fix responce when entering a non-existent user #7160

Merged
merged 1 commit into from
May 27, 2024
Merged

Conversation

KizerovDmitriy
Copy link
Contributor

@KizerovDmitriy KizerovDmitriy commented May 25, 2024

GreenCity PR

Summary Of Changes 🔥

  • add check, when name is empty string.
  • rewrote test logic.

Issue Link 📋

#6398

PR Checklist Forms

(to be filled out by PR submitter)

  • Code is up-to-date with the dev branch.
  • You've successfully built and run the tests locally.
  • There are new or updated unit tests validating the change.
  • JIRA/ Github Issue number & title in PR title (ISSUE-XXXX: Ticket title)
  • This template filled (above this section).
  • Sonar's report does not contain bugs, vulnerabilities, security issues, code smells or duplication
  • NEED_REVIEW and READY_FOR_REVIEW labels are added.
  • All files reviewed before sending to reviewers

Copy link

@KizerovDmitriy KizerovDmitriy self-assigned this May 27, 2024
@KizerovDmitriy KizerovDmitriy merged commit df57904 into dev May 27, 2024
3 checks passed
@KizerovDmitriy KizerovDmitriy deleted the bug-fix-#6398 branch May 27, 2024 09:57
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.

4 participants