Skip to content

Column creation mutation - support for "mirror" and "board_relation" column types + warnings#147

Open
Lewandowski-commits wants to merge 11 commits intoProdPerfect:masterfrom
Lewandowski-commits:master
Open

Column creation mutation - support for "mirror" and "board_relation" column types + warnings#147
Lewandowski-commits wants to merge 11 commits intoProdPerfect:masterfrom
Lewandowski-commits:master

Conversation

@Lewandowski-commits
Copy link

I've added the support for mirror and board_relation columns.

This is only supported on the current API version of 2026-01, so I've also updated the default API version in the client.

Since the creation of these columns is possible without the defaults that will actually create the links to specific boards/columns/items, I've also added warnings if a create_column mutation for these column types is ran without the key defaults for them.

Looks like there's already #132 for improving the queries on those column types, so both PRs could be potentially combined.

@Diosbejgli
Copy link
Collaborator

Can you add unit tests as well?


Available methods
^^^^^^^^^^^^^^^^^
**Available methods:**
Copy link
Collaborator

Choose a reason for hiding this comment

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

This way it'd become smaller, and less visible.

Choose a reason for hiding this comment

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

Comment on lines +35 to +38
Items Resource (monday.items)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
``create_item(board_id, group_id, item_name, column_values=None, create_labels_if_missing=False)``
- Create an item on a board in the given group with name item_name.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Any specific reason you're changing this?

Choose a reason for hiding this comment

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

Not sure why @olegkron made this change here - I've cherry-picked commits from his branch to include the tests for his work in my PR. Personally I'm good with not including the changes to README.rst

Copy link
Collaborator

@Diosbejgli Diosbejgli Mar 17, 2026

Choose a reason for hiding this comment

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

yeah, you can just exclude the README changes altogether.

Copy link
Collaborator

@Diosbejgli Diosbejgli left a comment

Choose a reason for hiding this comment

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

Can you add unit tests for your change?

@Lewandowski-commits
Copy link
Author

Can you add unit tests for your change?

Done in fca26ed. I've also cherry-picked the code from #106 and added tests for it as well here.


_DEFAULT_HEADERS = {
"API-Version": "2023-10"
"API-Version": "2026-01"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Any breaking changes?

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