Skip to content

Реализация setCustomReaderParams#2

Draft
mikekosulin wants to merge 8 commits into
Unact:masterfrom
mikekosulin:feat/custom-params
Draft

Реализация setCustomReaderParams#2
mikekosulin wants to merge 8 commits into
Unact:masterfrom
mikekosulin:feat/custom-params

Conversation

@mikekosulin

@mikekosulin mikekosulin commented Sep 16, 2023

Copy link
Copy Markdown

Настройка для автоматического включения NFC на ридере P17 при проведении транзакции

  • Android
  • iOS

@DCrow DCrow left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Спасибо за доработку!

Оставил замечания к PR.
Также остался вопрос насчет форматирования. Это с использованием dart format ?

Comment thread android/src/main/kotlin/com/unact/iboxpro_flutter/IboxproFlutterHandlerImpl.kt Outdated
Comment thread android/build.gradle Outdated
Comment thread lib/src/payment.dart Outdated
Comment thread example/android/settings.gradle Outdated
@mikekosulin

Copy link
Copy Markdown
Author

Оставил замечания к PR.

Окей, поправлю в течение недели, как будет время.

Также остался вопрос насчет форматирования. Это с использованием dart format ?

Да, через VSCode, могу без автоформатирования закоммитить

@DCrow

DCrow commented Dec 14, 2023

Copy link
Copy Markdown
Contributor

Да, через VSCode, могу без автоформатирования закоммитить

Да, лучше без него. У dart format довольно специфическое форматирование, иногда читать наоборот становится хуже.

@mikekosulin mikekosulin marked this pull request as draft December 18, 2023 21:40
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.

2 participants