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

Show "sign in to comment" link under all posts for anonymous users #1504

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

Conversation

n1313
Copy link
Contributor

@n1313 n1313 commented Feb 4, 2022

This PR adds "sign in to comment" link under all posts for anonymous users, and not just under "single" posts.

Before:

Screenshot 2022-02-03 at 20 06 03

After:
Screenshot 2022-02-03 at 20 06 38

@github-actions
Copy link

github-actions bot commented Feb 4, 2022

🎊 PR Preview e9615bd has been successfully built and deployed to https://FreeFeed-freefeed-react-client-preview-pr-1504.surge.sh

🕐 Build time: 260.38s

🤖 By surge-preview

@davidmz
Copy link
Member

davidmz commented Feb 4, 2022

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

@n1313
Copy link
Contributor Author

n1313 commented Feb 4, 2022

Для анонимуса в ленте постов вообще нет никаких call to action, призывов зарегистрироваться или залогиниться. Этот призыв есть только на индивидуальной странице поста. Я думаю, что добавление этих ссылок может привести к (небольшому) увеличению регистраций/логинов.

@davidmz
Copy link
Member

davidmz commented Feb 4, 2022

Может быть, правильнее показывать анонимусу стандартную кнопку Comment (под телом поста), но при нажатии предлагать авторизоваться? Так по крайней мере интерфейс будет единообразным

@n1313
Copy link
Contributor Author

n1313 commented Feb 4, 2022 via email

@davidmz
Copy link
Member

davidmz commented Feb 4, 2022

Наверное да. Хотя там как раз есть ссылочка внизу

@n1313
Copy link
Contributor Author

n1313 commented Feb 4, 2022 via email

@davidmz
Copy link
Member

davidmz commented Feb 5, 2022

Ну вроде бы моё предложение этому не противоречит.

Мне бы не хотелось, чтобы страница для анонимусов была шумнее чем для обычных юзеров. Вообще, все эти продуктовые завлекалки у меня вызывают опасения. Показать нормальный интерфейс — это пожалуйста, заодно анонимус увидит, что его ждёт после логина.

@n1313
Copy link
Contributor Author

n1313 commented Feb 5, 2022

Понимаю и принимаю твои аргументы. Давай обсудим это (и вообще стратегию взаимодействия с анонимусами) на следующем созвоне.

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