Add card assignees in card JSON builder #2118
Conversation
Jbuilder: Include assignees in card data
jorgemanrubia
left a comment
jorgemanrubia
left a comment
There was a problem hiding this comment.
This makes sense to me. What do you think @monorkin?
| json.board card.board, partial: "boards/board", as: :board | ||
| json.column card.column, partial: "columns/column", as: :column if card.column | ||
| json.creator card.creator, partial: "users/user", as: :user | ||
| json.assignees card.assignees, partial: "users/user", as: :user |
There was a problem hiding this comment.
Hey @petzbloom-sudo Nice work! I believe we could also update https://github.com/basecamp/fizzy/blob/main/docs/API.md?plain=1#L464, where there’s an example of the payload. What do you think?
|
@petzbloom-sudo @jorgemanrubia agreed, this makes sense. |
|
@petzbloom-sudo could we change your PR such that the partial returns the first few asignees, e.g. 3 or 5, and indicated if there are more on the show page? I did just merge a limit to the number of assignees on a card, but it's really high for an API response. |
4 participants
This change adds the "assignees" to the card.json.jbuilder partial.
Currently, the card JSON payload (used by both the API and Webhooks) includes the creator but omits assignment data. This makes it hard for webhook integrations to determine who is responsible for a card.
I have added the "assignees" collection to the standard card JSON response, reusing the existing "users/user" partial to ensure data consistency.
Note to Maintainers
I am a "low-code" contributor and do not have a local Rails environment set up to test this. I tried to update the webhook builder Json but it got closed so I'm trying again this way directly in the card.
Thanks,
Bloom