Skip to content

[63008] from_ field set by attribute on draft ignored#162

Merged
mrashed-dev merged 2 commits into
mainfrom
mostafarashed/ch63008/from-field-set-by-attribute-on-draft-ignored
Jul 9, 2021
Merged

[63008] from_ field set by attribute on draft ignored#162
mrashed-dev merged 2 commits into
mainfrom
mostafarashed/ch63008/from-field-set-by-attribute-on-draft-ignored

Conversation

@mrashed-dev
Copy link
Copy Markdown
Contributor

Description

from and in are reserved keywords, and to do circumvent this issue we add underscores after the attribute name. However, during the as_json method, we did not look for the underscored variable names.

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.

`from` and `in` are reserved keywords, and to do circumvent this issue we add underscores after the attribute name. However, during the as_json method, we did not look for the underscored variable names.
@shortcut-integration
Copy link
Copy Markdown

@mrashed-dev mrashed-dev linked an issue Jul 8, 2021 that may be closed by this pull request
@mrashed-dev mrashed-dev requested a review from philrenaud July 8, 2021 22:54
@mrashed-dev mrashed-dev merged commit 2d63f62 into main Jul 9, 2021
@mrashed-dev mrashed-dev deleted the mostafarashed/ch63008/from-field-set-by-attribute-on-draft-ignored branch July 9, 2021 13:14
@mrashed-dev mrashed-dev restored the mostafarashed/ch63008/from-field-set-by-attribute-on-draft-ignored branch July 12, 2021 16:25
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)
@mrashed-dev mrashed-dev deleted the mostafarashed/ch63008/from-field-set-by-attribute-on-draft-ignored branch August 30, 2023 21:57
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.

from_ field set by attribute on draft ignored when sending

2 participants