Skip to content

Remove competitors - #1004

Merged
fvacek merged 9 commits into
qe3from
remove-competitors
May 23, 2025
Merged

fvacek merged 9 commits into
qe3from
remove-competitors

Conversation

@fvacek

@fvacek fvacek commented May 5, 2025

Copy link
Copy Markdown
Collaborator

Competitors perspective removed, functionality moved to Runs:

  • competitors statistics report
  • set class in selected rows context menu
  • ranking_pos table column
  • new entries can be done using insert into table Runs

@fvacek
fvacek requested review from arnost00 and paukert May 5, 2025 21:12
@fvacek
fvacek force-pushed the remove-competitors branch from acbd3b3 to 71649b6 Compare May 8, 2025 20:29
@arnost00

This comment was marked as resolved.

@arnost00

This comment was marked as resolved.

@arnost00

This comment was marked as resolved.

@arnost00

arnost00 commented May 9, 2025

Copy link
Copy Markdown
Collaborator

Before merging, we need to update translations.

@fvacek

fvacek commented May 13, 2025

Copy link
Copy Markdown
Collaborator Author

It seems that the "Draw class" and "Remove drawing" buttons do not call the appropriate action. I was not able to find in the original code how the connection is implemented, whether it relies on automatic connection in Qt. It worked before the merge of runs and competitors.

Tested on action build & local build from this branch.

fixed in 8858cc4

Comment thread quickevent/app/quickevent/plugins/Runs/src/runstablemodel.cpp Outdated
arnost00

This comment was marked as resolved.

@fvacek

fvacek commented May 15, 2025

Copy link
Copy Markdown
Collaborator Author

How to set start time for new competitor in "Edit competitor" dlg? Field "Runs" is empty.

  1. Button Save will fill runs table, this should be automated somehow, now I don't know how.
  2. double click on start_time will open free (vacant) starting times to assign.

@arnost00

This comment was marked as resolved.

@fvacek

fvacek commented May 18, 2025

Copy link
Copy Markdown
Collaborator Author

In runs UI is missing menu item "Stations"->"Backup memory"

I'm not sure, if backup memory readout is or was working. Does it worth to keep it?

@arnost00

Copy link
Copy Markdown
Collaborator

In runs UI is missing menu item "Stations"->"Backup memory"

I'm not sure, if backup memory readout is or was working. Does it worth to keep it?

As i wrote earlier : I'm not sure about "Stations"->"Backup memory", maybe its old and unused.
Do we know of anyone who used this functionality?

@lukaskett

Copy link
Copy Markdown
Collaborator

I'm not sure, if backup memory readout is or was working. Does it worth to keep it?

As i wrote earlier : I'm not sure about "Stations"->"Backup memory", maybe its old and unused. Do we know of anyone who used this functionality?

We were using backup memory function few years ago. It can be used only for event in contact mode (contactless punch is not stored in the unit memory). There is no need to keep the function in my point of view.

@fvacek

fvacek commented May 19, 2025

Copy link
Copy Markdown
Collaborator Author

In runs table widget is missing column "IOF ID"

fixed in 185c02a

@fvacek

fvacek commented May 19, 2025

Copy link
Copy Markdown
Collaborator Author

After commit f7dcd70 I cannot add or remove competitor by [+] [-] buttons. obrazek

Button [-] in RunsWidget::editCompetitors ends on

		if(sel_rows.count() <= 1)
			return;

Button [+] in TableView::insertRow call emit editRowInExternalEditor, which is comented in RunsWidget::RunsWidget on line

// connect(ui->wRunsTableWidget->tableView(), &qfw::TableView::editRowInExternalEditor, this, &RunsWidget::editCompetitor, Qt::QueuedConnection);

fixed in 7273ff5

@arnost00 arnost00 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.

Nice work :)
Thanks

@fvacek
fvacek merged commit 7638f2f into qe3 May 23, 2025
@fvacek
fvacek deleted the remove-competitors branch June 8, 2025 14:45
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