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

Draft: RFC supporting single track repeat #1219

Draft
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

sarjann
Copy link

@sarjann sarjann commented Nov 12, 2023

Thinking of tackling the issue of single-track repeat not being supported.

Note I'm not planning on merging this branch as is as it's broken / logic / ugly doesn't work and doesn't implement the changes I'm discussing. This is more just to describe what I want to do.

I'm thinking of doing it in a similar way to that shown here if we want backward compatibility. i.e creating a separate field for repeat_one and handling the logic for that separately.

Or what I'd prefer to do is to make repeat an enum with something like Repeat::Track, Repeat::All, Repeat::Off although I'm pretty sure this will break backwards compatibility. Although I guess I could store it this way in the state but the messages/interface the end user sends can treat it as two different events and what's returned is repeat=true,repeat_one=false seperately for example, preserving backwards compatibility.

I'm new to this project, so sorry if I'm missing something or saying something stupid. Just wanted some feedback before going deeper. Thanks.

Related Issue: #19

@roderickvd roderickvd marked this pull request as draft November 14, 2023 19:44
@roderickvd
Copy link
Member

Converting to draft to emphasise what you're saying.

The cause of "repeat track" not working is described here: #19 (comment). Best development time spent would be to implement the dealer. Although I realise it would take quite some more hours and lines of code than such an intermediate thing.

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.

None yet

2 participants