Skip to content

⚗️ Migrade Demo SDK to KMP#73

Open
krllus wants to merge 28 commits intomigrate_demo_to_kmpfrom
sdk_v6/change_sdk_version
Open

⚗️ Migrade Demo SDK to KMP#73
krllus wants to merge 28 commits intomigrate_demo_to_kmpfrom
sdk_v6/change_sdk_version

Conversation

@krllus
Copy link
Contributor

@krllus krllus commented May 16, 2025

Descrição


Checklist

  • Testei as alterações localmente.
  • Atualizei qualquer documentação relevante.
  • Verifiquei que não há problemas de Lint ou formatação.
  • Confirmei que o código segue as diretrizes de contribuição do projeto.

krllus and others added 2 commits April 30, 2025 11:25
* adding send mail into list;

* bump libraries version because new SDK dependency;

* changing cancel map from itk to atk

* adjusts for email provider

* adding logs;

* adjusting email provider;

* adjusting email provider;

* add new sdk version

---------

Co-authored-by: italo.santana <italo.santana@stone.com.br>
@backstage-catalog-validator
Copy link

⚠️ Este repositório ainda não está catalogado no Backstage. ⚠️

Por favor, catalogue-o seguindo as instruções nesta documentação. [Via VPN].

💁 Qualquer problema ou dúvida, estamos no Slack, basta abrir um ticket no canal #help-foundation-platform.

@krllus krllus changed the title Sdk v6/change sdk version Migrade Demo SDK to KMP May 16, 2025
@krllus krllus changed the title Migrade Demo SDK to KMP ⚗️ Migrade Demo SDK to KMP May 16, 2025
@github-actions
Copy link

github-actions bot commented May 16, 2025

Dependency Review

✅ No vulnerabilities found.

Scanned Files

None

@krllus krllus marked this pull request as ready for review May 21, 2025 16:52
Text("Error: $code - $message")
Button(onClick = {
viewModel.initializeSDK(context, appInfo)
//viewModel.initializeSDK(context, appInfo)

Choose a reason for hiding this comment

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

pode ser removido esse comentario ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

acho que pode ser removido todo o componente do botao;
essa acao de inicializacao do SDK foi removida do viewmodel.


is TransactionEvent.CancelTransaction -> {
//providerWrapper.cancelTransaction()
// providerWrapper.cancelTransaction()

Choose a reason for hiding this comment

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

deveria ter alguma ação aqui ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

acho que esse cara se perdeu do merge que adicionou o ios; se nao me engano, o provider de reversao de transacoes com erro & provider de recibo estao nesse cenario;


val connectedDevice = bluetoothRepository.getConnectedBluetoothDevice()
if (connectedDevice == null) {
// TODO display error message indicating bluetooth is not connected

Choose a reason for hiding this comment

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

esse TODO já foi implementado ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ainda nao.

@marcostark marcostark requested a review from a team June 10, 2025 14:33
krllus and others added 8 commits June 12, 2025 10:31
* adjusting code because of update in SDK and HAL internal implementation;

* adjusting theme surface for dark mode;

* layout adjustments;

* layout adjustments;

* fix compilation error for email provider.

* change gradle configuration; trying to improve build time;

* adjusting navigation and components location

* removing log imports as they are not being used;

* fix paste and type of non digits in activation;

* adjustments in navigation between screens

* removing gertec from tasks to exclude when assemble
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.

3 participants