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.

Sign up for GitHub

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

Jump to bottom

Feature (Like): Add acessorio de view animado #5

Merged
merged 7 commits into from Aug 3, 2022
Merged

Feature (Like): Add acessorio de view animado #5

merged 7 commits into from Aug 3, 2022

Conversation

PaoloProdossimoLopes
Copy link
Contributor

@PaoloProdossimoLopes PaoloProdossimoLopes commented Jul 27, 2022 •
edited

📝 DESCRIÇÃO:

  • Add acessorio na lista
  • Animaçao no acessorio (fluxo normal e o inverso)

🔗 DOCUMENTAÇÃO ASSOCIADA: 

Task Notion

🖼 EVIDENCIAS

  • Não
  • Sim, Veja a baixo:
like-evidency.mp4

📍 COMO TESTAR:

FLUXO: BUILD APP
BRANCH: feature/lottie/like

☑️ CHECK LIST:

  • Eu revisei meu código?
  • Estou atualizado com a develop?
  • Removi os prints?
  • Estou seguindo o estilo do projeto?
  • Minhas mudanças não geraram alertas?
  • Eu validei meu layout em telas pequenas?
  • Eu validei meu layout em telas grandes?
  • Eu não dupliquei codigo!
  • Meu código não contem valores mágicos?
  • Meu código não contem textos perdidos?
  • Estou respeitando as nomenclaturas?
  • Eu não fiz nenhum 'Force unwrap' (!) perigoso
  • My code was reviewed by someone else

@PaoloProdossimoLopes
Feature (Like): Add acessorio de view animado
939c6c3 
Descriçao:
- Adicao do acessoryView
- Adicao da animacao no fluxo normal 0% -> 100% para like
- Adicao da aniamcao no fluxo reverso 100% -> 0% prara retirar-lo
@PaoloProdossimoLopes
Build: Resolve
2d3eb68
@PaoloProdossimoLopes
Refactor (ActivityCellView): Add constraints
76ff056
@PaoloProdossimoLopes
Feature (AnimationView): AnimationViews Customizadas baseadas na bibl…
981c0ab 
…ioteca do lottie
leocoout
leocoout reviewed Jul 28, 2022
View reviewed changes
.../FinanceApp/Commons/AnimationViews/Views/SingleLopAnimationView/SingleLopAnimationView.swift Outdated Show resolved Hide resolved
.../FinanceApp/Commons/AnimationViews/Views/SingleLopAnimationView/SingleLopAnimationView.swift Outdated Show resolved Hide resolved
.../FinanceApp/Commons/AnimationViews/Views/SingleLopAnimationView/SingleLopAnimationView.swift Outdated Show resolved Hide resolved
solutions/devsprint-leonardo-santos-2/FinanceApp/Components/ActivityCellView.swift Outdated Show resolved Hide resolved
solutions/devsprint-leonardo-santos-2/FinanceApp/Components/ActivityCellView.swift Outdated Show resolved Hide resolved
.../FinanceApp/Commons/AnimationViews/Views/SingleLopAnimationView/SingleLopAnimationView.swift Outdated Show resolved Hide resolved
@leocoout leocoout merged commit 99778e1 into devpass-tech:main Aug 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@leocoout leocoout leocoout left review comments

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

Successfully merging this pull request may close these issues.

None yet
2 participants
@PaoloProdossimoLopes @leocoout