-
-
Notifications
You must be signed in to change notification settings - Fork 4.5k
fix cors problems with newsfeed articles #2940
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
Conversation
Codecov Report
@@ 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
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
|
Seems to look good, but I'd like to wait for my approval until Tuesday when I return from my trip |
9dbd021 to
1be8f98
Compare
1be8f98 to
b00d758
Compare
rejas
left a comment
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.
just one small jsdoc addition needed :-)
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-OptionsorAccess-Control-Allow-Originheaders (as already mentioned in the docs).