codebattle icon indicating copy to clipboard operation
codebattle copied to clipboard

fix creator in tournaments

Open emp7yhead opened this issue 1 year ago • 7 comments

Fixes #1686

emp7yhead avatar Dec 27 '23 11:12 emp7yhead

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

ReDBrother avatar Dec 27 '23 14:12 ReDBrother

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

emp7yhead avatar Dec 27 '23 16:12 emp7yhead

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

emp7yhead avatar Dec 27 '23 16:12 emp7yhead

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

https://github.com/hexlet-codebattle/codebattle/blob/3d3d57f0977bb7085d2e5185754b580db8b1c1b5/services/app/apps/codebattle/assets/js/widgets/pages/tournament/TournamentHeader.jsx#L138

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

ReDBrother avatar Dec 27 '23 18:12 ReDBrother

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

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

ReDBrother avatar Jan 04 '24 01:01 ReDBrother

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

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

emp7yhead avatar Feb 07 '24 12:02 emp7yhead

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

ReDBrother avatar Feb 07 '24 13:02 ReDBrother