Skip to content

Conversation

@tomasr8
Copy link
Member

@tomasr8 tomasr8 commented Dec 8, 2023

Something that has been asked many times before is the ability to reference regform fields in the badge designer.
Currently you can use any regform as a source when printing badges and the list of available placeholders is limited to personal data since every regform has them.

The idea is to essentially allow linking templates to a specific regform. Once linked, you would be able reference any field from the linked regform.

Once linked, you have all registration data placeholders available in the badge designer and you can print the badges.

When printing badges/tickets, you can only select the linked regform (if any). The equivalent is true when configuring default ticket template in regforms (TicketsForm) - you can only select unlinked templates, or templates which are linked to the given regform.

This is what the UI for linking looks like currently:
image

For not currently linked templates, there is a 'plus' icon which opens dropdown where you can select to link a regform (same as when printing badges). To unlink, you click on the red 'X'.

@ThiefMaster
Copy link
Member

For double-sided:

  • Front/back side must be linked to the same regform (this implies both templates are linked to the event itself), or the backside must not be linked to anything
  • Linking a template to a regform should not be allowed if it's the backside of another template, unless that template is linked to the same registration form

@bpedersen2
Copy link
Contributor

Looks promising, if ready I can also try to review it in detail.

@tomasr8 tomasr8 force-pushed the designer-link-regform branch from 6bae113 to 4a999d2 Compare December 18, 2023 09:14
@tomasr8
Copy link
Member Author

tomasr8 commented Dec 18, 2023

Looks promising, if ready I can also try to review it in detail.

Hi! Yes, definitely feel free to give it a try :) It should be more or less working at this point, I mostly need to add tests.
I rebased it on the latest master and updated the migration so you just need indico db upgrade

@tomasr8 tomasr8 changed the title WIP: Allow linking badge templates to regforms Allow linking badge templates to regforms Dec 18, 2023
@tomasr8 tomasr8 marked this pull request as ready for review January 12, 2024 12:31
Copy link
Member

@duartegalvao duartegalvao left a comment

Choose a reason for hiding this comment

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

seems to work well!

@tomasr8 tomasr8 force-pushed the designer-link-regform branch 2 times, most recently from 133ec3d to 6ed84fd Compare January 25, 2024 09:58
@tomasr8 tomasr8 force-pushed the designer-link-regform branch 3 times, most recently from e0dddb3 to 6755199 Compare January 31, 2024 09:54
@ThiefMaster ThiefMaster force-pushed the designer-link-regform branch from 3021be1 to 38fceca Compare March 1, 2024 13:32
@ThiefMaster ThiefMaster force-pushed the designer-link-regform branch 2 times, most recently from 892573f to 87a693d Compare March 1, 2024 16:59
@ThiefMaster ThiefMaster added the alembic Contains database changes label Mar 9, 2024
@tomasr8 tomasr8 force-pushed the designer-link-regform branch from ef7d31c to f7f243f Compare March 14, 2024 14:56
@tomasr8 tomasr8 force-pushed the designer-link-regform branch from 06bd397 to 01b665c Compare March 15, 2024 10:27
@tomasr8 tomasr8 force-pushed the designer-link-regform branch 2 times, most recently from 92f6017 to e01d085 Compare March 15, 2024 17:03
@tomasr8 tomasr8 force-pushed the designer-link-regform branch from e01d085 to 1a256f0 Compare March 15, 2024 17:08
@ThiefMaster ThiefMaster enabled auto-merge (squash) March 15, 2024 17:09
@ThiefMaster ThiefMaster merged commit 7f6691b into indico:master Mar 15, 2024
@ThiefMaster ThiefMaster deleted the designer-link-regform branch March 15, 2024 17:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

alembic Contains database changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants