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

feat: add calendar getSchedule functionality #2914

Merged
merged 6 commits into from
Feb 7, 2024
Merged

feat: add calendar getSchedule functionality #2914

merged 6 commits into from
Feb 7, 2024

Conversation

Taaqif
Copy link
Contributor

@Taaqif Taaqif commented Jan 23, 2024

Category

  • Bug fix?
  • New feature?
  • New sample?
  • Documentation update?

Related Issues

fixes #2913

What's in this Pull Request?

Adds in calendar get schedule api endpoint

@bcameron1231
Copy link
Collaborator

bcameron1231 commented Jan 23, 2024

Hi! Thank you for this PR. A couple of modifications if you don't mind. :)

Please let us know if you have any questions.

@bcameron1231 bcameron1231 self-requested a review January 23, 2024 16:46
@Taaqif
Copy link
Contributor Author

Taaqif commented Jan 25, 2024

@bcameron1231 Thanks for the suggestions
I have applied these changes

Please let me know if this suits

Taaqif and others added 2 commits January 25, 2024 17:59
availabilityViewInterval request value should be a number
@bcameron1231
Copy link
Collaborator

bcameron1231 commented Jan 25, 2024

Looks good. Made one minor tweak to the request object.

Thank you for the PR!

packages/graph/calendars/types.ts Outdated Show resolved Hide resolved
packages/graph/calendars/types.ts Outdated Show resolved Hide resolved
packages/graph/calendars/types.ts Outdated Show resolved Hide resolved
@bcameron1231 bcameron1231 added this to the 3.23.0 milestone Jan 25, 2024
@juliemturner juliemturner merged commit d84f391 into pnp:version-3 Feb 7, 2024
1 check passed
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.

Add getSchedule for graph.calendar endpoint
3 participants