-
Notifications
You must be signed in to change notification settings - Fork 23
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
basic sharing for achievements #663
base: master
Are you sure you want to change the base?
Conversation
Страничка, на которую будет вести ссылка из соцсети, пока очень простая Что на ней нужно писать? (cc @irinfox @lasveritas) |
Мне в голову приходит только "Я получил бейдж за вступление в команду". Но.. это как-то не так задорно. Эта штука, кстати, кликабельна? На вики переадресует (и если да, то как это соотносится с тем, что ачивки не всем открыты?) |
Положил в тестинг (opencorpora.org:8080) |
@@ -38,7 +53,7 @@ | |||
|
|||
<script> | |||
$('.a-modal').on('shown', function() { | |||
$.post("/ajax/game_mark_shown.php"); | |||
// $.post("/ajax/game_mark_shown.php"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
А это зачем?
Ой. Забыл убрать - это чтобы новые ачивки нельзя было скрыть - попап показывается при каждой загрузке страницы. Надо будет убрать, когда натестируемся |
сейчас ачивка на этой страничке никуда не ведет, потому что вики ачивок доступно не всем. Можно действительно поделяшками серьезно заняться, когда включим ачивки на всех. Но если ты не авторизован, по идее, в вики ачивок ты тоже не попадаешь |
Так чего, это сейчас или потом? |
Мы принимали решение, что включаем поделяшки, после того, как включим 2015-07-19 15:58 GMT+04:00 Dmitry Granovsky notifications@github.com:
|
8fc9e67
to
16a59ed
Compare
Можно посмотреть на виртуалке, как будут выглядеть шарилки (пока они только в попапе).
В эту же ветку можно впиливать новые тексты