-
Notifications
You must be signed in to change notification settings - Fork 1
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: allow to query on multiple calendars of same user #30
feat: allow to query on multiple calendars of same user #30
Conversation
…d-integration-test-for-our-use-cases
…' into guillaume/feat/add-format-lint-via-biome
…e/refactor/convert-unix-timestamps-to-datetimes
…low query metadata
…' into guillaume/feat/add-js-docs-client-lib
…eat/allow-to-query-free-busy-multiple
describe('A user can be in groups', () => { | ||
it.todo('To be implemented') | ||
}) |
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.
Not handled by calendar service
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.
LGTM but one question:
@@ -111,6 +111,35 @@ pub mod get_event { | |||
pub type APIResponse = CalendarEventResponse; | |||
} | |||
|
|||
pub mod get_events_by_calendars { |
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.
A general question:
How do you plan to document the Rust code?
I don't know what the conventions are.
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.
Good question!
I admit ideally I should add comments (conventions)
I'll keep that in mind of the following PRs
Changed