Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add logo to event organizers #379

Closed
jfwiebe opened this issue Jan 18, 2017 · 16 comments
Closed

Add logo to event organizers #379

jfwiebe opened this issue Jan 18, 2017 · 16 comments

Comments

@jfwiebe
Copy link
Contributor

jfwiebe commented Jan 18, 2017

What's about adding a "logo" attribute to event organizers (similar to events) which is displayed on the front page of each event organizer?

@jfwiebe
Copy link
Contributor Author

jfwiebe commented Jan 18, 2017

(maybe even something like main color, similar to events)

@raphaelm
Copy link
Member

We have very few organizer-based settings, because that was not much of a priority to me so far, but I'd sure take pull requests :)

@bolutife-lawrence
Copy link
Contributor

bolutife-lawrence commented Mar 8, 2017

Hi guys,

I'm new to the project. Happy to contribute. Looks like I'll be taking a crack @ this one. I believe this should set me going in the right direction.

Also, forgot to mention that you guys did a great job with the doc. 😄

@raphaelm
Copy link
Member

raphaelm commented Mar 9, 2017

Thank you very much! :) If you have any questions just ask!

@bolutife-lawrence
Copy link
Contributor

Sure thing @raphaelm. From what I understand so far, we need to allow event organizers add logos to their organizer pages, and maybe a full-width banner to sit on (something that represents the organizer's' theme)? Also, maybe we could do the same for events because I noticed events don't have that feature as well. Am I understanding this correctly?

@raphaelm
Copy link
Member

raphaelm commented Mar 9, 2017

Yes. Let's go for the logo first in a similar way that we currently have event logos.

Once that works, if that's an area that you are interested in, I'd be happy to explore options to allow more custom styling together with you.

Currently we have logos (that can in theory be a full-width banner) and custom primary colors. However, it would be nice if even more theme customization was possible, I totally agree. It probably will not be that simple if we want to keep security and accessibility high, but I'm sure there are some things that can be improved.

@bolutife-lawrence
Copy link
Contributor

Sounds good. Getting right to it. 😄

@bolutife-lawrence
Copy link
Contributor

Looks like I'm missing a pretix.cfg file. I presume that's the highly confidential project's config file? I have been testing using the default DB (sqlite3).

@rixx
Copy link
Member

rixx commented Mar 9, 2017

The pretix.cfg structure is detailed here, development specific settings are mentioned here. Hope this helps :)

@raphaelm
Copy link
Member

raphaelm commented Mar 9, 2017

Why do you think you are missing it? :) Most development work should be possible without one. Let me know if we can improve on the documentation here!

@bolutife-lawrence
Copy link
Contributor

@raphaelm yeah you're right. It would not matter anyway since there are fallback options along the way in settings.py. Thanks @rixx, that would come in handy.

Right now, I'm getting this weird error:

django.db.utils.OperationalError: no such table: pretixmultidomain_knowndomain

I've tried resyncing my DB, making migrations, and re-migrating. None seems to work.

@bolutife-lawrence
Copy link
Contributor

bolutife-lawrence commented Mar 9, 2017

Works now. Had torm my local SQLite DB, re-added it, and ran the migrations again.

@raphaelm
Copy link
Member

raphaelm commented Mar 9, 2017

Okay, strange :( Glad to hear it works now.

@bolutife-lawrence
Copy link
Contributor

bolutife-lawrence commented Mar 10, 2017

Guys, what would be the best way to persist the logo_image value from the organizer's update form using the OrganizerSetting model? I noticed that these models(GlobalSetting, OrganizerSetting and EventSetting) are being used site-wide to hold key-value pairs of relevant data related to each entity. Since the OrganizerUpdate view class is using an UpdateView, would it be wise to create an extra attribute on the Organizer model or just use the already existing OrganizerSetting model?

@raphaelm
Copy link
Member

The OrganizerSetting can also hold a file as a value (though it is a bit of a hack), so we can use this, but yes, this will require that OrganizerUpdate contains both a ModelForm and a SettingsForm. You can just look at EventUpdate and copy from there -- it does exactly the same thing ;)

@bolutife-lawrence
Copy link
Contributor

"The OrganizerSetting can also hold a file as a value (though it is a bit of a hack)" -> yeah I figured :). Thanks a lot! This'll do.

@raphaelm raphaelm modified the milestone: 1.2.0 Mar 24, 2017
@raphaelm raphaelm removed this from the 1.2.0 milestone Apr 10, 2017
raphaelm pushed a commit that referenced this issue May 8, 2017
* [WIP] Add logo to event organizers.

* Fix indentation issues.

* Refactor code

Refactor code

Refactor code

* Add new migration

* Take files into account for organizer sform (settings form)

* Fix grammer

* Make bootstrap form errors specific to each fieldset

* Display logo on organizer's page

* Fix PR issues

Fix PR issues

Fix PR issues

* Reorder imports

* Remove conflicting migration

* Fix rebase conflict
aph3rson added a commit to aph3rson/pretix that referenced this issue May 11, 2017
Sync master with master of pretix/pretix@300f8f6

* Automatically sort new products to the end

* Drop "squash your commits" from the dev guide

* Add variation descriptions and allow to order addons

* Link to Django's runserver options in dev docs

* Allow <br> tags in rich text

* Copy from event: deal with deleted items

* Make validate_cart useful together with addons

* Fix collapsing panels in the addon choice step

* Button text change if addons are present

* Update translations

* Squash migrations and bump version

* Ticket PDFs: Do not hide attendee name if code is hidden

* Add a user guide on payments

* Link PayPal and Stripe documentation in the respective forms

* Hide payment fees if they are all equal to 0.00

* Refs pretix#39 -- New concept of "teams" (pretix#478)

* New models

* CRUD UI

* UI for adding/removing team members

* Log display for teams

* Fix invitations, move frontend

* Drop old models (incomplete)

* Drop more old stuff

* Drop even more old stuff

* Fix tests

* Fix permission test

* flake8 fix

* Add tests fore the new code

* Rebase migrations

* Fix typo in method name

* Update translations

* Force ordering of events on dashboard

* Fix typos in events

* Prepare the pretixdroid API for an async mode in the app

* Pretixdroid tests: Ignore microseconds (chopped by mysql)

* pretixdroid API: Add related lookups

* Add idempotenty nonces to pretixdroid API

* pretixdroid: force-accepting unpaids and time display

* Marked webfonts as binary files (pretix#487)

Webfonts now listed as binary in `.gitattributes`.
Works on pretix#486

* Fix pretix#456 -- Allow products to be excluded from ticket-generation (pretix#483)

* Added non-admission setting to event

`ticket_download_nonadm` now setting in storage. Still need logic for
order page/PDF generation.
Works on pretix#456.

* Download button considers `ticket_download_nonadm`

Modified Django tags to look at item admission attribute and
`ticket_download_nonadm` setting.
Works on pretix#456.

* Ticket output for non-admission disabled

PDFs/etc. will only be permitted/generated for items with the
`admission` attribute, or if the `ticket_download_nonadm` event setting
is true. Applies to single and whole-order ticket downloads.
Works on pretix#456.

* Fixed product exclusion in PDF output

Forgot PDF output was a plugin, now includes same check as base
`BaseTicketOutput.generate_order`.
Works on pretix#456

* Mail signature (pretix#485)

* added signature field -- no function yet

* added mail signature feature

* fixed style issue

* fixed problem with signature default

* added unit test for mail signatures

* added unit test for mail signatures

* [WIP] Fix pretix#447 -- Sendmail plugin: Create new mail based on an old one (pretix#476)

* send old email content to the new one

* error key event

* test commit

* query bad ID

* query bad ID

* query bad ID

* query bad ID

* Update pretixdroid API version

* Refs pretix#447 -- Extend copying old mails to subject and receipients

* Fixed bugs and added test for date range rendering (pretix#488)

* fixed bug for same dates, added unit check for daterange

* fixed local language override in unit test

* Fix pretix#297 -- pretixdroid: Show metrics in the control panel (pretix#481)

* add checkin status page

add dashboard widget
add checkin page under orders

* modify checkin logic

added new fields in checkin page
added filter items

* add tests for checkins & minor improvement

* support addin_product & noadm setting logic

* remove name ordering check test case

* Fix pretix#379 -- Add logo to event organizers (pretix#431)

* [WIP] Add logo to event organizers.

* Fix indentation issues.

* Refactor code

Refactor code

Refactor code

* Add new migration

* Take files into account for organizer sform (settings form)

* Fix grammer

* Make bootstrap form errors specific to each fieldset

* Display logo on organizer's page

* Fix PR issues

Fix PR issues

Fix PR issues

* Reorder imports

* Remove conflicting migration

* Fix rebase conflict

* Fix pretix#41 -- Drag-and-drop ticket editor

Undo/redo

Useful toolbox

Font selection

Add text content

Use hex for colors

JS-side dump and load

Save

Load layout, proper undo/redo

First steps to Python rendering

More PDF rendering

Copy and paste

Buttons for keyboard actions

Splash Screen

Block unbeforeunload in dirty state

Remove debugging output

Preview

Upload new PDFs via the editor

Fix bugs during PDF reload, link in settings form

New default ticket

Add OpenSans BI

Custom fonts, fix tests

* Added bootstrap-colorpicker

* Allow inline PDF display in CSP header

* Add fontpack to list of plugins

* Update German translation

* Add ticketoutputpdf's assets to MANIFEST.i

* Fix migration of old ticket styles

* Fix iCal download URL

* Multi-line location field, new field for admission time

* Admission date and time in editor

* Remove icon from "add to calendar"

* Try to fix PDF display problems in Safari

* Proxy cachedfiles that are used as editor previews

* Check Event.presale_is_running in more places

* Fix CSS generation with an empty color field

* Fix missing placeholders and reformat the sendmail view

* Fix bug that lead to wrong payment amount when switching payment method to PayPal later

* Update translation
raphaelm pushed a commit that referenced this issue May 17, 2017
* Syncing fork to upstream (#1)

Sync master with master of 300f8f6

* Automatically sort new products to the end

* Drop "squash your commits" from the dev guide

* Add variation descriptions and allow to order addons

* Link to Django's runserver options in dev docs

* Allow <br> tags in rich text

* Copy from event: deal with deleted items

* Make validate_cart useful together with addons

* Fix collapsing panels in the addon choice step

* Button text change if addons are present

* Update translations

* Squash migrations and bump version

* Ticket PDFs: Do not hide attendee name if code is hidden

* Add a user guide on payments

* Link PayPal and Stripe documentation in the respective forms

* Hide payment fees if they are all equal to 0.00

* Refs #39 -- New concept of "teams" (#478)

* New models

* CRUD UI

* UI for adding/removing team members

* Log display for teams

* Fix invitations, move frontend

* Drop old models (incomplete)

* Drop more old stuff

* Drop even more old stuff

* Fix tests

* Fix permission test

* flake8 fix

* Add tests fore the new code

* Rebase migrations

* Fix typo in method name

* Update translations

* Force ordering of events on dashboard

* Fix typos in events

* Prepare the pretixdroid API for an async mode in the app

* Pretixdroid tests: Ignore microseconds (chopped by mysql)

* pretixdroid API: Add related lookups

* Add idempotenty nonces to pretixdroid API

* pretixdroid: force-accepting unpaids and time display

* Marked webfonts as binary files (#487)

Webfonts now listed as binary in `.gitattributes`.
Works on #486

* Fix #456 -- Allow products to be excluded from ticket-generation (#483)

* Added non-admission setting to event

`ticket_download_nonadm` now setting in storage. Still need logic for
order page/PDF generation.
Works on #456.

* Download button considers `ticket_download_nonadm`

Modified Django tags to look at item admission attribute and
`ticket_download_nonadm` setting.
Works on #456.

* Ticket output for non-admission disabled

PDFs/etc. will only be permitted/generated for items with the
`admission` attribute, or if the `ticket_download_nonadm` event setting
is true. Applies to single and whole-order ticket downloads.
Works on #456.

* Fixed product exclusion in PDF output

Forgot PDF output was a plugin, now includes same check as base
`BaseTicketOutput.generate_order`.
Works on #456

* Mail signature (#485)

* added signature field -- no function yet

* added mail signature feature

* fixed style issue

* fixed problem with signature default

* added unit test for mail signatures

* added unit test for mail signatures

* [WIP] Fix #447 -- Sendmail plugin: Create new mail based on an old one (#476)

* send old email content to the new one

* error key event

* test commit

* query bad ID

* query bad ID

* query bad ID

* query bad ID

* Update pretixdroid API version

* Refs #447 -- Extend copying old mails to subject and receipients

* Fixed bugs and added test for date range rendering (#488)

* fixed bug for same dates, added unit check for daterange

* fixed local language override in unit test

* Fix #297 -- pretixdroid: Show metrics in the control panel (#481)

* add checkin status page

add dashboard widget
add checkin page under orders

* modify checkin logic

added new fields in checkin page
added filter items

* add tests for checkins & minor improvement

* support addin_product & noadm setting logic

* remove name ordering check test case

* Fix #379 -- Add logo to event organizers (#431)

* [WIP] Add logo to event organizers.

* Fix indentation issues.

* Refactor code

Refactor code

Refactor code

* Add new migration

* Take files into account for organizer sform (settings form)

* Fix grammer

* Make bootstrap form errors specific to each fieldset

* Display logo on organizer's page

* Fix PR issues

Fix PR issues

Fix PR issues

* Reorder imports

* Remove conflicting migration

* Fix rebase conflict

* Fix #41 -- Drag-and-drop ticket editor

Undo/redo

Useful toolbox

Font selection

Add text content

Use hex for colors

JS-side dump and load

Save

Load layout, proper undo/redo

First steps to Python rendering

More PDF rendering

Copy and paste

Buttons for keyboard actions

Splash Screen

Block unbeforeunload in dirty state

Remove debugging output

Preview

Upload new PDFs via the editor

Fix bugs during PDF reload, link in settings form

New default ticket

Add OpenSans BI

Custom fonts, fix tests

* Added bootstrap-colorpicker

* Allow inline PDF display in CSP header

* Add fontpack to list of plugins

* Update German translation

* Add ticketoutputpdf's assets to MANIFEST.i

* Fix migration of old ticket styles

* Fix iCal download URL

* Multi-line location field, new field for admission time

* Admission date and time in editor

* Remove icon from "add to calendar"

* Try to fix PDF display problems in Safari

* Proxy cachedfiles that are used as editor previews

* Check Event.presale_is_running in more places

* Fix CSS generation with an empty color field

* Fix missing placeholders and reformat the sendmail view

* Fix bug that lead to wrong payment amount when switching payment method to PayPal later

* Update translation

* Revert "Syncing fork to upstream (#1)"

This reverts commit 847d409.
Merged wrong, my bad.

* Formatted OTP secret

New variable `secretGrouped` in `2fa_confirm_totp.html`, user-friendly
version of OTP secret (split every 4 characters).
Works on #443.

* Adds manual secret entry OTP setup screen

`secretGrouped` exposed in user-friendly fashion. Includes short
instructions, copy-to-clipboard button, and js to hide instructions
unless user clicks on "Can't scan the barcode?" link.
Works on #443.

* Minor indentation issuer

Fixed indentation issue (L40).
Works on #443.

* Minor spacing issues

L265 of `user.py` failing flake8 tests, minor spacing fixes.

* Fixes indentation in `2fa_confirm_totp.html`

Per #490 (comment),
fixes an issue with indentation.
Works on #443, member of #490.

* Removed `aria-*` attributes

Per #490 (comment),
removes `aria` attributes from sub-tutorial.
Works on #443, member of #490.

* Pretix capitalization issue

Per #490 (comment),
fixes an issue with capitalization of pretix.
Works on #443, member of #490.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants