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

fix creator in tournaments #1703

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

Conversation

emp7yhead
Copy link
Contributor

Fixes #1686

@ReDBrother
Copy link
Collaborator

Фронтенд без сreatorId теперь подкосило в другом месте)

@emp7yhead
Copy link
Contributor Author

чет я проморгал получается)

@emp7yhead
Copy link
Contributor Author

Вроде все посмотрел и в упор не вижу, что может пойти не так. Направь меня, пожалуйста, в какую сторону смотреть )

@ReDBrother
Copy link
Collaborator

Вроде все посмотрел и в упор не вижу, что может пойти не так. Направь меня, пожалуйста, в какую сторону смотреть )

creatorId используется в вычислении флага canModerate, который отвечает за разрешение на модерацию турнира для админа и создателя турнира. Там скорее нужно искать по коду, а не по UI

@ReDBrother
Copy link
Collaborator

Короче, мне кажется, creator вообще не нужно палить наружу)

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

@emp7yhead
Copy link
Contributor Author

Давай откатим и просто уберем все упоминания о создателе в списках. Единственное, что добавим на клиент индикатор, что это мой или чужой турнир.

Так, я вроде вернулся. То есть на фронте убираю отображение создателя?
И каким индикатором показывать создатель или нет?

@ReDBrother
Copy link
Collaborator

Никак, мы не будем это показывать. На самой турнирной таблице мы просто будем передавать owner_id

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants