Skip to content

[61633] Add new Room Resource fields#156

Merged
mrashed-dev merged 6 commits into
mainfrom
mostafarashed/ch61633/add-new-room-resource-read-only-fields
Jun 15, 2021
Merged

[61633] Add new Room Resource fields#156
mrashed-dev merged 6 commits into
mainfrom
mostafarashed/ch61633/add-new-room-resource-read-only-fields

Conversation

@mrashed-dev
Copy link
Copy Markdown
Contributor

Description

Adds in the new Room Resource fields brought in with the v2.2 API update.

License

I confirm that this contribution is made under the terms of the MIT license and that I have the authority necessary to make this contribution on behalf of its copyright owner.

@shortcut-integration
Copy link
Copy Markdown

This pull request has been linked to Clubhouse Story #61633: Add new Room Resource read only fields.

@yusra-ahmed-zz
Copy link
Copy Markdown

Do you need to add any tests for these?

@mrashed-dev
Copy link
Copy Markdown
Contributor Author

Do you need to add any tests for these?

There weren't any changes needed to these tests but it's a good idea to add some. Will add and request review when it's done, thank you!

Comment thread tests/test_resources.py


@pytest.mark.usefixtures("mock_resources")
def test_resource(api_client):
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@yusra-ahmed It's not much but the endpoint is a very basic GET endpoint without an query support!

Copy link
Copy Markdown

@yusra-ahmed-zz yusra-ahmed-zz left a comment

Choose a reason for hiding this comment

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

lgtm

@mrashed-dev mrashed-dev merged commit c4a564a into main Jun 15, 2021
@mrashed-dev mrashed-dev deleted the mostafarashed/ch61633/add-new-room-resource-read-only-fields branch June 15, 2021 20:05
mrashed-dev added a commit that referenced this pull request Jul 21, 2021
New `nylas` v5.0.0 release bringing in the following additions:
* Add support for the Nylas Neural API (#163)
* Add `metadata` support (#152)
* Add new Room Resource fields  (#156)
* Add `Nylas-API-Version` header support (#157, #151)

as well as the following changes:
* Transitioned from `app_id` and `app_secret` naming to `client_id` and `client_secret` (#159, #86)
* Fix adding a tracking object to an existing `draft` (#153)
* Fix issue when converting offset-aware `datetime` objects to `timestamp` (#154, #143)
* Fix `limit` value in filter not being used when making `.all()` call (#155)
* Fix `from_` field set by attribute on draft ignored (#162, #160)
* Remove `bumpversion` from a required dependency to an extra dependency (#158, #144)
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