Skip to content

Conversation

@khassel
Copy link
Collaborator

@khassel khassel commented Oct 8, 2022

solves #2840 as far as possible. There could still be errors on the embedded iframe when the owner of the site has set X-Frame-Options or Access-Control-Allow-Origin headers (as already mentioned in the docs).

@codecov-commenter
Copy link

codecov-commenter commented Oct 8, 2022

Codecov Report

Merging #2940 (4f562d5) into develop (835c893) will increase coverage by 1.22%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop    #2940      +/-   ##
===========================================
+ Coverage    64.06%   65.28%   +1.22%     
===========================================
  Files            9       14       +5     
  Lines          295      726     +431     
===========================================
+ Hits           189      474     +285     
- Misses         106      252     +146     
Impacted Files Coverage Δ
js/node_helper.js 78.04% <0.00%> (ø)
modules/default/calendar/calendarutils.js 56.20% <0.00%> (ø)
modules/default/calendar/calendarfetcher.js 80.32% <0.00%> (ø)
js/class.js 63.88% <0.00%> (ø)
modules/default/calendar/node_helper.js 54.28% <0.00%> (ø)
js/server.js 67.56% <0.00%> (+1.35%) ⬆️
js/app.js 70.24% <0.00%> (+9.91%) ⬆️
js/fetch.js 83.33% <0.00%> (+66.66%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@rejas
Copy link
Collaborator

rejas commented Oct 9, 2022

Seems to look good, but I'd like to wait for my approval until Tuesday when I return from my trip

Copy link
Collaborator

@rejas rejas left a comment

Choose a reason for hiding this comment

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

just one small jsdoc addition needed :-)

@khassel khassel requested a review from rejas October 17, 2022 18:55
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