Skip to content

Conversation

@SergioGlorias
Copy link
Member

Sometimes we need to offer support, to organizers without delay and we cannot be limited

@SergioGlorias
Copy link
Member Author

The tag has been implemented, but it needs a filter so it doesn't appear yet

@SergioGlorias
Copy link
Member Author

image

@SergioGlorias SergioGlorias marked this pull request as ready for review April 14, 2024 00:47
@SergioGlorias SergioGlorias changed the title Add private tier Broadcast: Add private tier Apr 16, 2024
ornicar added 3 commits April 16, 2024 17:13
* master:
  add lila.ui to better organize templating across modules
  move some tutor templates to lila.tutor.ui
  reorder modules in build.sbt
  remove study dep to analyse
  fix insights perf db type
  fix puzzle service initialization order
  fix msg service initialization order
  fix puzzle service initialization order
  lila.tree.Analyser WIP
  z-index tricks
  only embed lpvs on auto link
and I don't think it's worth caching it either
@ornicar
Copy link
Collaborator

ornicar commented Apr 16, 2024

looking pretty good. I still have to check that private broadcasts can't be found by other players.

@SergioGlorias
Copy link
Member Author

SergioGlorias commented Apr 16, 2024

looking pretty good. I still have to check that private broadcasts can't be found by other players.

I think it can be detected on the broadcaster account
and the studies of the creator

@ornicar
Copy link
Collaborator

ornicar commented Apr 17, 2024

This is basically useless until the broadcasts studies are made private. Which means:

  • turning all existing studies private when switching to private
  • turning all studies public the other way around
  • setting the study visibility when an additional broadcast round is created

@SergioGlorias
Copy link
Member Author

I think we can leave this configuration to broadcasters to decide what is best for each study
but it already solves other issues that we might need from this tier

ornicar added 3 commits April 17, 2024 13:15
* master:
  let kids read lichess blog posts - closes #15092
  Revert annotator test
  Make GameToRoot an object
@ornicar ornicar merged commit 32fe942 into lichess-org:master Apr 17, 2024
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