-
Notifications
You must be signed in to change notification settings - Fork 172
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
Meetings #739
Merged
Merged
Meetings #739
Changes from all commits
Commits
Show all changes
32 commits
Select commit
Hold shift + click to select a range
2512e65
Document meetings.complete
392d545
Add change to changelog
bb65621
Merge branch 'master' into ATL-2280-meeting-complete
joaoinacio 5ea8063
Document meetings.delete
ba837a7
document meetings.update
joaoinacio 3f95b79
add to changelog
joaoinacio 5c35a01
chore| add meeting.schedule
tiagobastos 287d67e
chore| fix bad indentation on assignees
tiagobastos 166e0e9
fix: use correct attendees property name
joaoinacio afdf7f2
add the location request parameter to meetings.update
joaoinacio 375c79a
id is required for location type contact/company
joaoinacio 13b3e93
Merge pull request #744 from teamleadercrm/ATL-2262-meetings_update
joaoinacio d5468f1
Merge remote-tracking branch 'origin/ATL-2280-meeting-complete' into …
tiagobastos bc89b7c
chore| fix attendees property & order they are displayed (for consist…
tiagobastos 12294d8
chore| add location
tiagobastos e9b91db
fix| use correct wording
tiagobastos f7585f4
fix| /meetings.info location address
tiagobastos ad6a1bf
Merge pull request #746 from teamleadercrm/meetings.info
joaoinacio 390ca9d
Merge pull request #745 from teamleadercrm/meetings.schedule
joaoinacio b7e9b1d
Merge remote-tracking branch 'origin/master' into ATL-2280-meeting-co…
tiagobastos 0064877
fix| return meeting id on schedule
tiagobastos 3f3d0df
chore| add finishing touches on meetings.* endpoints
tiagobastos da3a292
Document meeting web hooks
17ec30e
Provide meeting recurrence id
joaoinacio 7777c61
Rename meeting.added to meeting.created
c471bba
Merge pull request #754 from teamleadercrm/ATL-2259-meetings-webhooks
3d15196
Merge branch 'ATL-2280-meeting-complete' into ATL-1251_meeting_recurr…
71a3373
Merge pull request #757 from teamleadercrm/ATL-1251_meeting_recurrence
tiagobastos c9a8102
Document recurrence_id filter on meetings.list
joaoinacio 24bb806
Merge pull request #759 from teamleadercrm/ATL-2321_meetings_list_rec…
tiagobastos 0f1b4ac
Merge remote-tracking branch 'origin/master' into ATL-2280-meeting-co…
tiagobastos 6ba2f43
chore| move changes to current month
tiagobastos File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Q: We don't expose this resource anywhere yet, do we?
To which type refers this id then? Without exposing the resource I think we shouldn't include this property yet.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤔 Right. the goal is to be able to list recurring meetings from a recurrence id, so providing the recurrence (definition) isn't necessary for now.
We can omit the type property for now and provide just the id, which is the necessary info to have.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay, where would that id then come from? From existing meetings which return the recurrence?
Though, I wouldn't really remove the type if we'd still expose the id.