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

AND-7983 [Staking] Implemented enter staking Polkadot #3846

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

iMaks99
Copy link
Contributor

@iMaks99 iMaks99 commented Nov 29, 2024

@iMaks99 iMaks99 requested a review from a team as a code owner November 29, 2024 11:27
Copy link

Pull reviewers stats

Stats of the last 30 days for tangem-app-android:

User Total reviews Time to review Total comments
kozarezvlad 80 1h 1m 15
iMaks99 77 2h 40m 12
nzeeei 62 1h 33m 6
Yoggam1 43 59m 1
MasterBin 39 42m 8
Sateetas 27 1h 4m 14
Mama1emon 27 1h 28m 26
iiiburnyiii 6 1h 35m 9
nemelianov-tangem 3 6m 0
EvgeniiKuzovkin 3 10m 0
thebrodyaga 1 3m 0

⚡️ Pull request stats

@@ -88,6 +89,10 @@ internal object NetworkModule {
context = context,
apiConfigsManager = apiConfigsManager,
clientBuilder = {
this.callTimeout(STAKE_KIT_API_TIMEOUT_SECONDS, TimeUnit.SECONDS)
Copy link
Contributor

Choose a reason for hiding this comment

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

мб заюзаем наш класс Timeouts?
плюс эти 4 строки где-то в соседнем апи тоже используются, можно
вынести проставление таймаутов + обработку аннотаций в отдельный метод/экстеншн-метод с параметром timeoutSeconds

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