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

[Refatoração] Ajustes no Gerenciamento de Áudio e Downgrade do flutter_sound #279

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

RoxyRoses
Copy link
Contributor

[Refatoração] Ajustes no Gerenciamento de Áudio e Downgrade do flutter_sound

Descrição

Este pull request atualiza a lógica de gerenciamento de áudio nas classes de serviços para adequar-se à versão utilizada do pacote flutter_sound. Além disso, realiza o downgrade do pacote flutter_sound para a versão 8.5.0, corrigindo inconsistências com a versão anterior e audio que não estava sendo gravado corretamente.

Alterações Principais

  • Substituição de métodos obsoletos ou incompatíveis:

    • openPlayer substituído por openAudioSession no serviço de reprodução.
    • closePlayer substituído por closeAudioSession no serviço de reprodução.
    • openRecorder substituído por openAudioSession no serviço de gravação.
    • closeRecorder substituído por closeAudioSession no serviço de gravação.
  • Inserção de comentários com marcação TODO:

    • Os métodos originais foram comentados e marcados para revisão futura após o upgrade do Flutter e do pacote flutter_sound.
  • Alteração na dependência flutter_sound:

    • Downgrade da versão no pubspec.yaml de ^9.2.13 para 8.5.0.

Motivação

As alterações visam manter a compatibilidade do projeto com a versão atual do Flutter e resolver problemas de execução relacionados ao flutter_sound. A adoção dos novos métodos reflete a API disponível na versão 8.5.0.

Checklist

  • Dependências atualizadas no pubspec.yaml.
  • Substituição de métodos para openAudioSession e closeAudioSession.
  • Adição de comentários TODO para revisões futuras após upgrade do Flutter e flutter_sound.
  • Testes manuais realizados para verificar reprodução e gravação de áudio.
  • Logs de erros revisados para capturar exceções corretamente.

Observações

  • Após a atualização para uma versão mais recente do Flutter e do pacote flutter_sound, os métodos marcados como TODO devem ser revisados e substituídos pelos métodos originais.
  • Todas as mudanças foram testadas e não impactam outras funcionalidades do aplicativo.

@RoxyRoses RoxyRoses self-assigned this Dec 2, 2024
@RoxyRoses RoxyRoses requested a review from a team as a code owner December 2, 2024 15:04
@diraol
Copy link
Contributor

diraol commented Dec 2, 2024

@RoxyRoses você consegue liberar uma versão de testes (uma para cada SO, claro)?

Aí já peço para a equipe testar

@RoxyRoses
Copy link
Contributor Author

@RoxyRoses você consegue liberar uma versão de testes (uma para cada SO, claro)?

Aí já peço para a equipe testar

Consigo sim mas nao consigo mandar os apks por aqui, por onde mando?

@diraol
Copy link
Contributor

diraol commented Dec 2, 2024

@RoxyRoses você consegue liberar uma versão de testes (uma para cada SO, claro)?
Aí já peço para a equipe testar

Consigo sim mas nao consigo mandar os apks por aqui, por onde mando?

Nós temos um fluxo automatizado aqui no github para isso :)
Basicamente fazer o deploy no ambiente de staging já deveria fazer isso.

E os APKs são enviados via firebase/flightest se não me engano

@diraol diraol had a problem deploying to firebase-distribution December 2, 2024 17:58 — with GitHub Actions Failure
@diraol diraol temporarily deployed to firebase-distribution December 2, 2024 17:58 — with GitHub Actions Inactive
@RoxyRoses
Copy link
Contributor Author

@RoxyRoses você consegue liberar uma versão de testes (uma para cada SO, claro)?
Aí já peço para a equipe testar

Consigo sim mas nao consigo mandar os apks por aqui, por onde mando?

Nós temos um fluxo automatizado aqui no github para isso :) Basicamente fazer o deploy no ambiente de staging já deveria fazer isso.

E os APKs são enviados via firebase/flightest se não me engano

Legal, obrigada!

@diraol
Copy link
Contributor

diraol commented Dec 2, 2024

@RoxyRoses você consegue liberar uma versão de testes (uma para cada SO, claro)?
Aí já peço para a equipe testar

Consigo sim mas nao consigo mandar os apks por aqui, por onde mando?

Nós temos um fluxo automatizado aqui no github para isso :) Basicamente fazer o deploy no ambiente de staging já deveria fazer isso.
E os APKs são enviados via firebase/flightest se não me engano

Legal, obrigada!

Deixei indicado no Telegram como fazer :)

@diraol
Copy link
Contributor

diraol commented Dec 2, 2024

(Deu erro no iOS)

@RoxyRoses RoxyRoses force-pushed the downgrade_flutter_sound branch from ab28d8c to fe31b7f Compare December 2, 2024 20:27
@RoxyRoses RoxyRoses had a problem deploying to firebase-distribution December 2, 2024 23:03 — with GitHub Actions Failure
@RoxyRoses RoxyRoses temporarily deployed to firebase-distribution December 2, 2024 23:03 — with GitHub Actions Inactive
Copy link

sonarcloud bot commented Dec 3, 2024

Copy link

codeclimate bot commented Dec 3, 2024

Code Climate has analyzed commit f86a3a8 and detected 1 issue on this pull request.

Here's the issue category breakdown:

Category Count
Complexity 1

View more on Code Climate.

@RoxyRoses RoxyRoses temporarily deployed to firebase-distribution December 3, 2024 15:35 — with GitHub Actions Inactive
@RoxyRoses RoxyRoses temporarily deployed to firebase-distribution December 3, 2024 15:35 — with GitHub Actions Inactive
@RoxyRoses
Copy link
Contributor Author

(Deu erro no iOS)

Corrigido, agora gerou a versão para testes.

@RoxyRoses RoxyRoses deployed to firebase-distribution December 11, 2024 14:59 — with GitHub Actions Active
@RoxyRoses RoxyRoses temporarily deployed to firebase-distribution December 11, 2024 14:59 — with GitHub Actions Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants