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

Авторизация, восстановление сессии, установка токена, правка api доков, NSCoding моделей #395

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

ivabra
Copy link

@ivabra ivabra commented Apr 17, 2016

Раз Сделал возможным принудительно не открывать приложение для авторизации, даже если оно есть. Для этого добавил в перечисление VKAuthorizationOptions элемент VKAuthorizationOptionsDisableApp.

Два Добавил в wakeUpSession параметр для восстановления сессии без интернета.

+ (void)wakeUpSession:(NSArray *)permissions useInternetToUpdateSession: (BOOL) useInternet completeBlock:(void (^)(VKAuthorizationState, NSError *))wakeUpBlock;

Старый метод работает по прежнему.

Три Открыл метод [VKSdk setAccessToken:] для ручной установки токена.

Четыре Добавил в VKApi доступ к api доков. Добавил в модель доков новое свойство type.

Пять Все модели имплементят класс NSCoding. Для того, чтобы токен правильно восстанавливался (вместе с localUser), потребовалось, чтобы VKUser имплементил NSCoding. По коду было удобнее сделать это с VKApiObject.

@RomanTruba
Copy link
Contributor

В чём проблема с приложением?

@ivabra
Copy link
Author

ivabra commented Apr 17, 2016

@Dreddik кейс когда хочется войти под другим аккаунтом, нежели в офф клиенте.

@RomanTruba
Copy link
Contributor

Это плохая практика, надо убрать

@ivabra
Copy link
Author

ivabra commented Apr 17, 2016

Мне было бы интересно услышать аргументы в пользу этой самой плохой практики.
Спорить не буду, просто интересно.

@RomanTruba
Copy link
Contributor

Вынуждая пользователя игнорировать приложение, вы теряете пользователей/игроков, потому что людям нужно всё время вводить логин/пароль. Авторизация через приложение и задумывалась для того, чтобы легко и в пару нажатий залогиниться в приложении или игре.
Когда-нибудь мы сделаем в приложении возможность перелогиниться быстро, но пока нет

@ivabra
Copy link
Author

ivabra commented Apr 17, 2016

В приложении предполагалось сделать выбор - через приложение либо через браузер. Причём второй выбор в виде маленькой подписи.
Зачем лишать пользователей такой возможности.

@@ -35,6 +35,7 @@
static NSString *const HTTPS_REQUIRED = @"https_required";
static NSString *const CREATED = @"created";
static NSString *const PERMISSIONS = @"permissions";
static NSString *const LOCAL_USER = @"local_user";
Copy link
Contributor

Choose a reason for hiding this comment

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

Что это?

@4031651
Copy link

4031651 commented Aug 4, 2016

По поводу кейса с принудительным открытием в сафари.
Ещё не знаю как на iOS, но на андроиде есть проблема со старыми приложениями. Старых приложений ещё довольно много. Кто-то отключил обновление, а у кого-то закончилось место. Проблема уходит с установкой свежей версии приложения VK. На скрине можно видеть старый дизайн.
tempfileforshare_2016-08-03-14-12-17

По этому поводу есть ишью в репе vk-android-sdk: VKCOM/vk-android-sdk#101

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