Skip to content

Add quick setup to call for abstracts notifications - #6454

Merged
ThiefMaster merged 9 commits into
indico:masterfrom
jbtwist:abstract-quick-setup
Sep 3, 2024
Merged

Add quick setup to call for abstracts notifications#6454
ThiefMaster merged 9 commits into
indico:masterfrom
jbtwist:abstract-quick-setup

Conversation

@jbtwist

@jbtwist jbtwist commented Jul 25, 2024

Copy link
Copy Markdown
Contributor

No description provided.

@jbtwist
jbtwist force-pushed the abstract-quick-setup branch 2 times, most recently from e708c52 to 3f437f4 Compare July 25, 2024 13:15
@jbtwist
jbtwist marked this pull request as ready for review July 25, 2024 13:16
Comment thread indico/modules/events/abstracts/blueprint.py Outdated
Comment thread indico/modules/events/abstracts/controllers/email_templates.py Outdated
Comment thread indico/modules/events/models/events.py Outdated
Comment thread indico/modules/events/abstracts/templates/management/_notification_tpl_list.html Outdated
Comment thread indico/modules/events/abstracts/models/abstracts.py Outdated
Comment thread indico/modules/events/abstracts/controllers/management.py Outdated
Comment thread indico/modules/events/abstracts/controllers/email_templates.py Outdated
@jbtwist
jbtwist force-pushed the abstract-quick-setup branch 2 times, most recently from 726183a to b73bc40 Compare August 6, 2024 14:18
Comment thread indico/modules/events/abstracts/controllers/management.py Outdated
Comment thread indico/modules/events/abstracts/templates/management/_notification_tpl_list.html Outdated
Comment thread indico/modules/events/models/events.py Outdated
@jbtwist
jbtwist force-pushed the abstract-quick-setup branch 2 times, most recently from c1bf3eb to 310b62b Compare August 13, 2024 09:18
@ThiefMaster
ThiefMaster force-pushed the abstract-quick-setup branch from 310b62b to 1e4f95e Compare August 20, 2024 09:43
@jbtwist
jbtwist force-pushed the abstract-quick-setup branch 4 times, most recently from 82e32ba to def74b2 Compare August 26, 2024 16:00
@jbtwist
jbtwist force-pushed the abstract-quick-setup branch from def74b2 to 332edeb Compare September 2, 2024 11:53
@jbtwist
jbtwist marked this pull request as draft September 2, 2024 13:51
@jbtwist
jbtwist force-pushed the abstract-quick-setup branch from bae6aec to da4a452 Compare September 2, 2024 14:07
@jbtwist
jbtwist marked this pull request as ready for review September 2, 2024 14:07
Comment thread indico/modules/events/abstracts/templates/management/_notification_tpl_list.html Outdated
@jbtwist
jbtwist force-pushed the abstract-quick-setup branch from da4a452 to e4f1d03 Compare September 3, 2024 13:29
- no underscores in urls
- missing i18n
- only render notification html when needed
- better notification template titles + no "magic" to convert titles to
  template names
Pressing enter in the title field triggered that button instead of the
submit button
@ThiefMaster ThiefMaster added this to the v3.3 milestone Sep 3, 2024
@ThiefMaster
ThiefMaster enabled auto-merge (squash) September 3, 2024 14:22
@ThiefMaster
ThiefMaster merged commit 47b8ff0 into indico:master Sep 3, 2024
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