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

feat(bux-399): Confirm contact #497

Merged
merged 6 commits into from
Mar 13, 2024
Merged

Conversation

Nazarii-4chain
Copy link
Contributor

@Nazarii-4chain Nazarii-4chain commented Mar 11, 2024

Added confirm Contact PIKE

Pull Request Checklist

  • πŸ“– I created my PR using provided : CODE_STANDARDS
  • πŸ“– I have read the short Code of Conduct: CODE_OF_CONDUCT
  • 🏠 I tested my changes locally.
  • βœ… I have provided tests for my changes.
  • πŸ“ I have used conventional commits.
  • πŸ“— I have updated any related documentation.
  • πŸ’Ύ PR was issued based on the Github or Jira issue.

@Nazarii-4chain Nazarii-4chain self-assigned this Mar 11, 2024
@Nazarii-4chain Nazarii-4chain requested a review from a team as a code owner March 11, 2024 07:43
Copy link

github-actions bot commented Mar 11, 2024

Manual Tests

πŸ’š Manual testing by @arkadiuszos4chain resulted in success.

@Nazarii-4chain Nazarii-4chain changed the title Bux 399/confirm contact feat(bux-399): Confirm contact Mar 11, 2024
actions/contacts/models.go Show resolved Hide resolved
actions/contacts/models.go Show resolved Hide resolved
actions/contacts/models.go Show resolved Hide resolved
actions/contacts/routes.go Show resolved Hide resolved
engine/action_contact.go Show resolved Hide resolved
actions/contacts/update.go Show resolved Hide resolved
@codecov-commenter
Copy link

codecov-commenter commented Mar 12, 2024

Codecov Report

Attention: Patch coverage is 31.37255% with 35 lines in your changes are missing coverage. Please review.

Project coverage is 53.09%. Comparing base (e88eba0) to head (64c7fc7).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #497      +/-   ##
==========================================
+ Coverage   52.97%   53.09%   +0.12%     
==========================================
  Files         164      165       +1     
  Lines        9027     9076      +49     
==========================================
+ Hits         4782     4819      +37     
- Misses       3793     3799       +6     
- Partials      452      458       +6     
Flag Coverage Ξ”
unittests 53.09% <31.37%> (+0.12%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Ξ”
actions/contacts/routes.go 100.00% <100.00%> (ΓΈ)
engine/model_contact.go 70.10% <78.94%> (+22.60%) ⬆️
actions/contacts/update.go 0.00% <0.00%> (ΓΈ)
engine/action_contact.go 0.00% <0.00%> (ΓΈ)

... and 1 file with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Ξ” = absolute <relative> (impact), ΓΈ = not affected, ? = missing data
Powered by Codecov. Last update e88eba0...64c7fc7. Read the comment docs.

@arkadiuszos4chain arkadiuszos4chain added the tested PR was tested by a team member label Mar 13, 2024
@arkadiuszos4chain arkadiuszos4chain added tested PR was tested by a team member and removed tested PR was tested by a team member labels Mar 13, 2024
@arkadiuszos4chain arkadiuszos4chain merged commit a5563e0 into master Mar 13, 2024
6 of 7 checks passed
@arkadiuszos4chain arkadiuszos4chain deleted the BUX-399/ConfirmContact branch March 13, 2024 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tested PR was tested by a team member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants