Skip to content

Refactored code.| #1574#1815

Merged
IvanPizhenko merged 3 commits intomasterfrom
issue_1574_improve_code_viewing
May 30, 2022
Merged

Refactored code.| #1574#1815
IvanPizhenko merged 3 commits intomasterfrom
issue_1574_improve_code_viewing

Conversation

@DenBond7
Copy link
Collaborator

This PR refcatored code.

close #1574


Tests (delete all except exactly one):

  • Does not need tests (refactor only, docs or internal changes)

To be filled by reviewers

I have reviewed that this PR... (tick whichever items you personally focused on during this review):

  • addresses the issue it closes (if any)
  • code is readable and understandable
  • is accompanied with tests, or tests are not needed
  • is free of vulnerabilities

@DenBond7 DenBond7 marked this pull request as ready for review May 29, 2022 14:54
@DenBond7 DenBond7 requested a review from IvanPizhenko as a code owner May 29, 2022 14:54
IvanPizhenko
IvanPizhenko previously approved these changes May 29, 2022
@IvanPizhenko
Copy link
Contributor

@DenBond7 merge conflict here

DenBond7 added 2 commits May 30, 2022 09:38
# Conflicts:
#	FlowCrypt/src/main/java/com/flowcrypt/email/database/dao/MessageDao.kt
@IvanPizhenko IvanPizhenko merged commit 3388064 into master May 30, 2022
@IvanPizhenko IvanPizhenko deleted the issue_1574_improve_code_viewing branch May 30, 2022 10:31
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.

AndroidStudio can't syntax-highlight multiple line SQL - report to maintainers

2 participants