Skip to content

Searching relay runners by Bib, when using assign card to runner. - #1198

Open
VojtechVelat wants to merge 1 commit into
Quick-Box:mainfrom
VojtechVelat:fix/relay-find-runner-bib
Open

Searching relay runners by Bib, when using assign card to runner.#1198
VojtechVelat wants to merge 1 commit into
Quick-Box:mainfrom
VojtechVelat:fix/relay-find-runner-bib

Conversation

@VojtechVelat

Copy link
Copy Markdown

When organizing a relay it is not possible to search runners by Bib, when using the "assign card to runner". All bibs default to 0.
image

This commit fixes this problem.
image

Comment thread aqtinstall.log

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

should not be part of commit IMO

.joinRestricted("competitors.id", "runs.competitorId", "runs.stageId=" QF_IARG(stage_id), "JOIN")
.orderBy("classes.name, lastName, firstName");
if(is_relays) {
qb.select("relays.number AS \"competitors.startNumber\"")

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
qb.select("relays.number AS \"competitors.startNumber\"")
qb.select("relays.number AS startNumber")

.join("relays.classId", "classes.id");
}
else {
qb.select2("competitors", "startNumber")

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
qb.select2("competitors", "startNumber")
qb.select("competitors.startNumber AS startNumber")

@arnost00

Copy link
Copy Markdown
Collaborator

I see that another feature has been added - displaying the start number and leg in the Card Readout.

obrazek

@arnost00

Copy link
Copy Markdown
Collaborator

Thanks for this PR. It helps with relays.

@fvacek
fvacek self-requested a review August 25, 2026 18:46

@fvacek fvacek left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

any progress?

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