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

Добавлена возможность авторизации через token #47

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

xTCry
Copy link

@xTCry xTCry commented Sep 7, 2020

No description provided.

@danyadev
Copy link
Owner

danyadev commented Sep 7, 2020

Я и сам был бы рад возможности входить по любому токену, однако для адекватной работы многих функций просто необходим доступ к закрытому апи, который доступен только через токен вк для андроид.

К тому же, токен должен иметь доступ к разделу сообщений, то есть быть от официального приложения.

Я могу разрешить вход по токену VK для Android, потому что это не создаст никаких проблем, но не более того.

А еще я нашел у тебя багу в коде - ты забыл убрать (или подставить туда access_token) поле android_token

image

@xTCry
Copy link
Author

xTCry commented Sep 7, 2020

Конечно лучше скрыть поле с токеном по дефолту и выводить его при шорткоде или аргументом при запуске прил.

Авторизация через токен может быть как доп фичей этого приложения (можно создать отдельную ветвь).
Если токен будет использован от обычного VK приложения или VK Me, то это же вполне допустимо

Да, не особо вник в код и сделал это на скорую руку
Как будет время, то побольше разберусь и разовью эту тему

@danyadev
Copy link
Owner

danyadev commented Mar 9, 2021

@xTCry Есть желание доработать?

@xTCry
Copy link
Author

xTCry commented Mar 9, 2021

@danyadev

@xTCry Есть желание доработать?

Да, как будет время.
Какой именно функционал?

@danyadev
Copy link
Owner

@xTCry как минимум стоит подогнать код к текущему состоянию кода репозитория

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