Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Contact + Message API fixes #1

Open
wants to merge 28 commits into
base: master
Choose a base branch
from

Conversation

jpcamara
Copy link
Member

@jpcamara jpcamara commented Dec 11, 2018

Adding source to contact to better support contact api
Overriding serialize method for labels so saving a message works (it requires a list of ids, so when we save and it gives us a list of hashes it blows up)
Adds tracking to drafts
to_h always returns the attribute value as long as it's not nil, so we can clear fields in an update
Don't save before sending draft to send directly
Removed unused web_page from contacts
Exclude sending object in when when saving an event

Overriding serialize method for labels so saving a message works (it requires a list of ids, so when we save and it gives us a list of hashes it blows up)
Use same type for folder and labels
@jpcamara jpcamara force-pushed the message-and-contact-api-fixes branch from 35b8adf to 9f8db50 Compare December 18, 2018 18:10
@Che-Chris Che-Chris force-pushed the message-and-contact-api-fixes branch from 5b24ab4 to cb66650 Compare January 14, 2019 13:58
@Che-Chris Che-Chris force-pushed the message-and-contact-api-fixes branch 2 times, most recently from b823396 to a018d0c Compare July 10, 2019 16:15
@jpcamara jpcamara force-pushed the message-and-contact-api-fixes branch from a018d0c to a2f77b6 Compare July 10, 2019 16:18
* This is to hot fix a production issue where a field that is supposed to be a list is sometimes coming back as a string
jpcamara and others added 4 commits May 18, 2020 10:29
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