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

Contemplando projeto com upgrade de frameworks #4

Open
wants to merge 6 commits into
base: 1.1
Choose a base branch
from

Conversation

msoares94
Copy link

Esse PR contempla a atualização do projeto com as versões mais recentes do Ionic e Angular

@msoares94
Copy link
Author

msoares94 commented Sep 16, 2024

@edersoares conforme prometido, segue o PR com o upgrade do projeto para contemplar as versões mais recentes dos frameworks (Ionic e Angular)

Após aceitar o PR, é necessário ajustar o arquivo src/environments/environment.prod.ts para trabalhar com o carregamento dos ambiente via requisição na API.

@msoares94 msoares94 changed the title Contemplanto projeto com upgrade de frameworks Contemplando projeto com upgrade de frameworks Sep 16, 2024
@edersoares
Copy link
Member

@msoares94 que contribuição incrível! Muito obrigado por isso.

Vamos adicionar no nosso fluxo de testes internos e mantenho você informado!

Incrível!

Copy link
Member

@edersoares edersoares left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@msoares94 fiz alguns testes e pude notar alguns ajustes, vou fazendo por partes para que possam ser corrigidos gradativamente.

Sugiro termos nos arquivos environment.ts e environment.prod.ts variáveis de ambiente que sejam mais fáceis de substituir ao invés de mexer diretamente no código.

src/app/sign-in/sign-in.page.ts Outdated Show resolved Hide resolved
src/app/sign-in/sign-in.page.ts Show resolved Hide resolved
@leoholiveira
Copy link

leoholiveira commented Oct 9, 2024

@msoares94 ótima contribuição. Muito boa a sua iniciativa e achei excelente essa contribuição.

Sobre o app, notei alguns pontos que chamou minha atenção:

  • As versões das bibliotecas "capacitor" encontram uma vulnerabilidade que permite injection;
image
  • Quando efetuei o login com um usuário e depois desloguei e entrei com outro, o app mantem os dados do usuário que foi efetuado o login pela primeira vez (acredito que seja um problema de cache);

  • O backend retornou dados de frequência, porém no app não efetou o carregamento da lista (pode ser algum parametrô que não consegui encontrar).

image

Sobre a atualização no geral, está muito boa mesmo, agradeço novamente.

@msoares94
Copy link
Author

@msoares94 ótima contribuição. Muito boa a sua iniciativa e achei excelente essa contribuição.

Sobre o app, notei alguns pontos que chamou minha atenção:

* As versões das bibliotecas "capacitor" encontram uma vulnerabilidade que permite injection;
image
* Quando efetuei o login com um usuário e depois desloguei e entrei com outro, o app mantem os dados do usuário que foi efetuado o login pela primeira vez (acredito que seja um problema de cache);

* O backend retornou dados de frequência, porém no app não efetou o carregamento da lista (pode ser algum parametrô que não consegui encontrar).

image

Sobre a atualização no geral, está muito boa mesmo, agradeço novamente.

Corrigido, obrigado pelos apontamentos

@msoares94
Copy link
Author

msoares94 commented Dec 19, 2024

@edersoares, @leoholiveira e mais quem for interessado, vamos unir esforços para fazermos esse PR ser aceito, a versão atual do Android para a versão 1.1 já não é mais aceita pelas lojas de aplicativo.

Esse PR foi um trabalho de comunidade, entendo que pode não estar 100% da forma como a Portabilis deseja, mais pode ser trabalhado em conjunto para fecharmos uma nova versão.

@ferox
Copy link

ferox commented Dec 26, 2024

@edersoares Tendo em vista que todos os apontamentos e correções foram realizados, esta PR está apta para entrar no repositório.

Há algo a mais que precisa ser alterado?

@leoholiveira Chegou a testar o login e a frequência após os ajustes?

@edersoares
Copy link
Member

@msoares94 obrigado novamente pela contribuição.

Eu vou tocar este projeto diretamente, já estou fazendo melhorias e refatorações para que seja possível mantermos o aplicativo atualizado.

Nós vamos precisar de apoio nos testes então peço a todos os interessados que quiserem fazer parte destes testes me avisem.

Nós fizemos a atualização para o Ionic 8 e Angular 18 (19 não está permitindo build pra Android) e iniciei a reescrita do core da sincronização que está uma bagunça. Minha ideia é ter uma versão disponível para testes na semana que vem.

@msoares94
Copy link
Author

Olá @edersoares que bom que pude ajudar!
Fico disponível como sempre estou para tratar desses assuntos.

Pode me colocar como interessado nos testes também.

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.

4 participants