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(#58): Adiciona controle de visualizacao onboarding #65

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

dylancavalcante
Copy link
Contributor

…-APP#58)

Descrição

Este pull request modifica a lógica de controle de contagem de visualizações no fluxo de onboarding do aplicativo, conforme previsto na issue fga-eps-mds/2024.2-ARANDU-DOC#58. As alterações incluem a otimização do código para incrementar a contagem de visualizações e a correção da integração com o armazenamento local de dados.

Revisão

  • A lógica do contador de visualizações foi corrigida para garantir que o valor correto seja armazenado e recuperado do armazenamento seguro (StorageValue).
  • A função incrementAndGetViewCount() foi substituída pela chamada direta a incrementNumber() no OnboardingViewModel para simplificar o fluxo de execução.

Checklist

  • O conteúdo adicionado nesse PR segue o código de conduta do projeto;
  • Foram seguidos os padrões estabelecidos no guia de contribuição;
  • O conteúdo adicionado foi devidamente testado;
  • O conteúdo do PR foi revisado por membro alheio ao mesmo;
  • O conteúdo deste PR foi devidamente identado, comentado e documentado;
  • Para conteúdos de documentação, foi adicionada tabela de versionamento aos artefatos.

Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@dylancavalcante dylancavalcante changed the title [FEAT] Implementa funcionalidade ViewCount (fga-eps-mds/2024.2-ARANDU… feat(#58): Adiciona controle de visualizacao onboarding Jan 10, 2025
zDrNz added a commit that referenced this pull request Jan 28, 2025
MarinaGaldi added a commit that referenced this pull request Jan 30, 2025
GabrielCostaDeOliveira added a commit that referenced this pull request Jan 30, 2025
zDrNz added a commit that referenced this pull request Jan 31, 2025
Yasm1nNasc1mento added a commit that referenced this pull request Feb 1, 2025
Yasm1nNasc1mento added a commit that referenced this pull request Feb 1, 2025
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.

1 participant