Skip to content

Freebusy support#11

Open
shazam442 wants to merge 6 commits intopat:mainfrom
digineo:freebusy
Open

Freebusy support#11
shazam442 wants to merge 6 commits intopat:mainfrom
digineo:freebusy

Conversation

@shazam442
Copy link

No description provided.

spec.add_runtime_dependency "nokogiri"
spec.add_dependency "http"
spec.add_dependency "icalendar"
spec.add_dependency "nokogiri"
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mistake... switch back to add_runtime_dependency

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rubocop lint fix


def call
Nokogiri::XML::Builder.new do |xml|
xml["c"].public_send(:"free-busy-query",
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe use: ??

xml["caldav"].public_send(:"free-busy-query", NAMESPACES) do

@pat
Copy link
Owner

pat commented Aug 2, 2025

@shazam442 sorry it's been such a wait for me to look at this - and I realise now there's a little bit of overlap with the work in #12.

Do you feel your work is ready here? Is it possible to rebase it, and ideally remove the CI changes - they could be a separate PR, let's keep this one focused on the functionality change if possible.

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