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

bugfix: presenter emails are not showing up on the meeting detail page. #527

Open
JoeJasinski opened this issue Apr 11, 2024 · 3 comments
Open

Comments

@JoeJasinski
Copy link
Contributor

In the admin of the meeting detail page, we currently have a readonly field that should display the emails of all approved meeting presenters, for easy copy/paste into an email. This is useful for contacting the presenters before the meeting.

https:/chicagopython/chipy.org/blob/main/chipy_org/apps/meetings/admin.py#L116

That field appears to be empty now, even for events with approved presenters.

image

This is the method that generates that list which might need adjustments, or perhaps the way django sanitizes this type of input has changed between versions.

https:/chicagopython/chipy.org/blob/main/chipy_org/apps/meetings/models.py#L185

@JoeJasinski
Copy link
Contributor Author

JoeJasinski commented Apr 11, 2024

I believe something changed between versions. Here's the new syntax.
https://docs.djangoproject.com/en/5.0/ref/contrib/admin/#django.contrib.admin.ModelAdmin.readonly_fields

@JoeJasinski
Copy link
Contributor Author

I got this working. See the related PR

@JoeJasinski
Copy link
Contributor Author

image

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

1 participant