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

Unobtrusive login #99

Merged
merged 22 commits into from
Apr 18, 2017
Merged

Unobtrusive login #99

merged 22 commits into from
Apr 18, 2017

Conversation

meanmail
Copy link
Collaborator

@meanmail meanmail commented Apr 17, 2017

Who should approve:

Solves issue(s):
IDEA-221
IDEA-223
IDEA-224
Description (and images):

  1. Больше не мучаем пользователь почти бесконечными запросами залогиниться. Пользователю больше не обязательно быть авторизованным (конечно он не может отправлять решения и т.д.).
  2. Диалог для логина показываем только, когда пользователь явно производит действие, требующее его авторизации.
  3. Если пользователь отказывается залогинится, то действие не производиться;
  4. Для анонима показывается в Step Description кнопка "Login";
  5. После события "Пользователь авторизовался" или "Пользователь вышел" автоматически:
  • Обновляется Step Description (для анонима показать кнопку "Login");
  • Обновляется прогресс у видимых элементов в дереве проекта (для анонима сбросить прогресс);
  • Обновляется имя пользователя в Settings, Project Wizard (если эти окна открыты);
  • Если пользователь залогинился, то производиться обновление/синхронизацию проекта.
  1. При создании проекта аноним не сможет пройти дальше шага с выбором курса (При нажатии "Next" показывается диалог для логина, если пользователь не залогинится, то ему сообщаем о том, что логин обязателен для продолжения). На сам же шаг с выбором курса пускаем всех пользователей без обязательного логина.
  2. При открытии новой рекомендации в адаптивных курсах закрываются все файлы предыдущей рекомендации.
  3. Теперь все необработанные исключения JavaScript из браузера перенаправляются в лог приложения. А так же все вызовы console.log и т.д.
  4. Блокируются кнопки "Send", "Download submissions", "Reset" для всех шагов кроме Code-quiz, так как они в других степах не используются.
  5. Устранен возможный deadlock и NPE.
  6. При смене содержимого Step Description и выполнении действий показывается анимация.

@meanmail meanmail added this to the R-0.15 milestone Apr 17, 2017
@meanmail meanmail self-assigned this Apr 17, 2017
@meanmail meanmail requested a review from taery April 17, 2017 20:23
@meanmail meanmail changed the base branch from feature/free_anwer to develop April 18, 2017 07:48
@meanmail meanmail merged commit 26ab7cc into develop Apr 18, 2017
@meanmail meanmail deleted the feature/unobtrusive_login branch April 18, 2017 07:48
@meanmail meanmail added this to the R-0.15 milestone May 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant