Skip to content

expose sponsor contact for pycon#2984

Merged
JacobCoffee merged 1 commit intomainfrom
sponsor-contact-api
Apr 15, 2026
Merged

expose sponsor contact for pycon#2984
JacobCoffee merged 1 commit intomainfrom
sponsor-contact-api

Conversation

@JacobCoffee
Copy link
Copy Markdown
Member

Description

  • Expose sponsor contacts for use with PyCon staff dashbaords

Copilot AI review requested due to automatic review settings April 15, 2026 23:43
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: bf31b4cef0

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread apps/sponsors/api.py
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds sponsor contact details to the sponsors “logo placement” DRF endpoint so PyCon staff dashboards can access contact info alongside sponsor/logo metadata.

Changes:

  • Introduces SponsorContactSerializer and nests it under LogoPlacementSerializer as contacts.
  • Extends LogoPlacementeAPIList response to include sponsor contacts (name/email + role flags).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
apps/sponsors/serializers.py Adds a serializer for sponsor contact data and exposes it as a nested field on the logo placement payload.
apps/sponsors/api.py Builds and returns contacts data per sponsor within the logo placement API response.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread apps/sponsors/api.py
Comment thread apps/sponsors/api.py
Comment thread apps/sponsors/api.py
@JacobCoffee JacobCoffee merged commit 5d5338f into main Apr 15, 2026
18 checks passed
@JacobCoffee JacobCoffee deleted the sponsor-contact-api branch April 15, 2026 23:51
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.

2 participants