-
Notifications
You must be signed in to change notification settings - Fork 4
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(task): delete task #56
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dá uma olhada nesse pontos
.DS_Store
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Acho q esse arquivo nn deveria ter subido
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Acabei de ver aqui, é um arquivo do Mac OS, adicione no gitignore pfv e remova ele do pr
feature/logbook/src/main/java/com/loryblu/feature/logbook/ui/home/LogbookHomeViewModel.kt
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Boa!
@duhdoesk podemos fazer o merge? |
Task
This pull request refers to task LOR-XX
Description
Please include a summary of the change and which issue is fixed.
Types of changes
What changed:
Checklist
development
(if not - rebase it or merge it).If any of the acceptance criteria are not met, please explain why below
What is the criteria? Why it is different?