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

simplify replace system #16721

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

Conversation

SergioGlorias
Copy link
Member

not need 2 systems

image
Screenshot 2025-01-04 00 40 17

Note: I noticed that fide ID placed (except 0) does not allow changing names and classification
but let's leave it like that
the important thing was the other tools non-standard events

Target: #content-dev>player name replacements

Comment on lines +53 to 54
// Original name / Optional FideID / Optional title / Optional rating / Optional replacement name
private def parse(line: String): Option[(PlayerName, RelayPlayerLine)] =
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wouldn't this change break all previous broadcasts replace?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes, it will break the previous Broadcasts a little...

but it also doesn't make sense to have 2 different systems (if 1 is used, it will alert broadcasters)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we could do what happened with the change of the old system: leave "a month" with the system running 2 systems and then delete the old one

@ornicar
Copy link
Collaborator

ornicar commented Jan 6, 2025

I agree the simplification is good.

Let's see if it breaks past broadcasts. These functions are called:

  • during the broadcast sync
  • when the broadcast form is saved

So I don't think past broadcasts would be affected, unless their form is submitted again for some other updates.
What happens then, are the players infos lost, does the form refuse to submit, ...?

@SergioGlorias
Copy link
Member Author

in this case, previous broadcasts that are taking place
Without a transition period, previous transmissions stop being correct
meaning it will not replace any broadcasts
in other words: everything that was made has no adornment until it is replaced

for official broadcasts, the normal thing to do is just use player name = fide id
We rarely use the other option, except for special events when we have to replace the name

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.

3 participants