Skip to content

feat: ofeed service improvements - #1104

Merged
fvacek merged 21 commits into
Quick-Box:mainfrom
lukaskett:feature/ofeed
Apr 6, 2026
Merged

feat: ofeed service improvements#1104
fvacek merged 21 commits into
Quick-Box:mainfrom
lukaskett:feature/ofeed

Conversation

@lukaskett

@lukaskett lukaskett commented Mar 31, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator
  • added test connection button
  • added paste setup link - set credentials in one step - event id and password
  • added show password button
  • added open event page button
  • extended additional settings of the service and general one as well:
    • add qr code with a link to the event - appends class url param with competitor's class, can be configured in the settings or ofeed service, always bottom
    • add event logo/image from OFeed or loaded local file in the settings with adjusting the height, always top

OFeed service
image

Upgraded Receipts settings
image

Receipt example
image

#992 #1063

lukaskett added 13 commits March 31, 2026 23:55
@lukaskett
feat: OFeed service upgrade
542a3bc 
Added test button
Normalized hostUrl
When inserting into table with changes:
- changed status -> Accepted
- source -> OFeed
@lukaskett
feat: add custom image/logo to the receipt from OFeed when available …
1b18b73 
…with height adjustment

# Conflicts:
#	quickevent/app/quickevent/quickevent-cs_CZ.ts
#	quickevent/app/quickevent/quickevent-fr_FR.ts
#	quickevent/app/quickevent/quickevent-nb_NO.ts
#	quickevent/app/quickevent/quickevent-nl_BE.ts
#	quickevent/app/quickevent/quickevent-pl_PL.ts
#	quickevent/app/quickevent/quickevent-ru_RU.ts
#	quickevent/app/quickevent/quickevent-uk_UA.ts
@lukaskett
feat: add show/hide button for the password value
e612ab7
@lukaskett
feat: add Paste setup link button
bb9283b
@lukaskett
feat: adjust base url
a114d2e
@lukaskett
feat: Add QR code option, adds a new dependency which should be maybe…
786864a 
… placed in 3party in root
@lukaskett
feat: extend qr code link with competitor's class
6cedac7
@lukaskett
feat: Add open event page button
84d87e3
@lukaskett
rename config keys for qr code and image/logo
6280cdb
@lukaskett
feat: add support for qr code and event logo/image to the receipt in …
36f6ee6 
…the same way as it works for ofeed
@lukaskett
fix: remove migration login and generalize code in receiptsplugin
63dd589
@lukaskett
feat: append competitor's class to the link
6cff654
@lukaskett
fix: sync settings and ofeed service setup - qr code and logo/image
dccb9d3
@lukaskett
lukaskett requested review from arnost00 and fvacek April 1, 2026 07:33
arnost00
arnost00 reviewed Apr 1, 2026
View reviewed changes
Comment thread libqf/libqfgui/src/model/logtablemodel.h Resolved
fvacek
fvacek requested changes Apr 2, 2026
View reviewed changes
Comment thread quickevent/app/quickevent/plugins/Receipts/qml/reports/receipts/Classic.qml Resolved
Comment thread quickevent/app/quickevent/plugins/Receipts/qml/reports/receipts/Default.qml Resolved
Comment thread quickevent/app/quickevent/plugins/Receipts/src/receiptssettingspage.cpp Resolved
Comment thread quickevent/app/quickevent/plugins/Receipts/src/receiptssettingspage.cpp Resolved
@fvacek
fvacek self-requested a review April 2, 2026 20:07
fvacek
fvacek requested changes Apr 2, 2026
View reviewed changes

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

I would have clang-tidy warnings resolved, we have some options:

  1. Fix QRC generator code
  2. Remove qrcodegen.* from clang-tidy checked files

@lukaskett

lukaskett commented Apr 2, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator Author

I realized the qr code caption like Live Results could be useful as well. I'll try to add it as well. Please wait with merging, it's in process.

@lukaskett

lukaskett commented Apr 2, 2026

Copy link
Copy Markdown
Collaborator Author

I would have clang-tidy warnings resolved, we have some options:

  1. Fix QRC generator code
  2. Remove qrcodegen.* from clang-tidy checked files

I applied option 2 because it's third-party code, and option 1 would likely require maintaining own fork.

@lukaskett
lukaskett requested a review from fvacek April 4, 2026 21:49
@lukaskett

lukaskett commented Apr 6, 2026

Copy link
Copy Markdown
Collaborator Author

20260404_224218

@fvacek
fvacek merged commit c06cc70 into Quick-Box:main Apr 6, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@arnost00 arnost00 arnost00 left review comments
@fvacek fvacek Awaiting requested review from fvacek

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@lukaskett @fvacek @arnost00