Bug: update SD dance level auto-detect to allow MS26 and Plus26

Closed
  • Is something wrong?
#1,767 2 comments 0 reactions 1 assignee View on GitHub

@mpogue2 is already working on this.

Since Sep 30, 2026.

Assessment

This issue has not been assessed yet.

Description

bug Ready for final check

SD built into SquareDesk supports two new levels: "mainstream26" and "plus26".

Currently, if a dance is named "foo_ms26" The "ms26" is not recognized, and I believe it will be auto-detected as "plus",
when it now should be auto-detected as "mainstream26".

So, going forward,
dance name == "mainstream" --> set dance program to "mainstream"
dance name == "ms26" --> set dance program to "mainstream26"
dance name == "plus" --> set dance program to "plus26"

where dance name = the last field using FS="_".

Check the code to make sure that I am describing this correctly, before making changes.

Dominant language
C++
Stars
14
Forks
4
No merged PRs in 30d

Getting set up

This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from mpogue2/SquareDesk

All issues in mpogue2/SquareDesk

Similar issues

More C++ issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.