Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

UBER-560: filter out current transaction and get mixin #3480

Merged
merged 2 commits into from
Jul 4, 2023
Merged

UBER-560: filter out current transaction and get mixin #3480

merged 2 commits into from
Jul 4, 2023

Conversation

ThetaDR
Copy link
Contributor

@ThetaDR ThetaDR commented Jul 4, 2023

Contribution checklist

Brief description

Checklist

  • - UI test added to added/changed functionality?
  • - Screenshot is added to PR if applicable ?
  • - Does the code work? Check whether function and logic are correct.
  • - Does Changelog.md is updated with changes?
  • - Does the translations are up to date?
  • - Does it well tested?
  • - Tested for Chrome.
  • - Tested for Safari.
  • - Go through the changed code looking for typos, TODOs, commented LOCs, debugging pieces of code, etc.
  • - Rebase your branch onto master and upstream branch
  • - Is there any redundant or duplicate code?
  • - Are required links are linked to PR?
  • - Does new code is well documented ?

Related issues

https://front.hc.engineering/workbench/platform/tracker/UBER-560
A list of closed updated issues

@ThetaDR
UBER-560: filter out current transaction and get mixin
2f382e1 
Signed-off-by: Vyacheslav Tumanov <me@slavatumanov.me>
@ThetaDR ThetaDR requested a review from BykhovDenis July 4, 2023 13:58
@ThetaDR
UBER-560: format fix
b6e05f4 
Signed-off-by: Vyacheslav Tumanov <me@slavatumanov.me>
@haiodo haiodo merged commit a0ecb3c into main Jul 4, 2023
@haiodo haiodo deleted the UBER-560 branch July 4, 2023 18:45
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@haiodo haiodo haiodo approved these changes

@BykhovDenis BykhovDenis BykhovDenis approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ThetaDR @haiodo @BykhovDenis