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: allow to query on multiple calendars of same user #30

Merged

Conversation

GuillaumeDecMeetsMore
Copy link
Collaborator

@GuillaumeDecMeetsMore GuillaumeDecMeetsMore commented Aug 7, 2024

Changed

  • Implemented "get events of multiple calendars for one user"

GuillaumeDecMeetsMore and others added 30 commits July 11, 2024 11:32
…' into guillaume/feat/add-format-lint-via-biome
…e/refactor/convert-unix-timestamps-to-datetimes
@GuillaumeDecMeetsMore GuillaumeDecMeetsMore self-assigned this Aug 7, 2024
Base automatically changed from guillaume/feat/allow-to-query-free-busy-multiple to master August 19, 2024 09:06
@GuillaumeDecMeetsMore GuillaumeDecMeetsMore marked this pull request as ready for review August 27, 2024 05:00
Comment on lines -379 to -381
describe('A user can be in groups', () => {
it.todo('To be implemented')
})
Copy link
Collaborator Author

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

Copy link

@mm-zacharydavison mm-zacharydavison left a 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 {

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.

Copy link
Collaborator Author

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

@GuillaumeDecMeetsMore GuillaumeDecMeetsMore merged commit ed01502 into master Aug 28, 2024
2 checks passed
@GuillaumeDecMeetsMore GuillaumeDecMeetsMore deleted the guillaume/feat/allow-to-query-multiple-calendars branch August 28, 2024 01:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants