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

add Newsblur #107

Open
wants to merge 16 commits into
base: master
Choose a base branch
from
Open

add Newsblur #107

wants to merge 16 commits into from

Conversation

lioman
Copy link

@lioman lioman commented Sep 4, 2018

Description and issue

I've added Newsblur as Feedreader

List of significant changes made

  • add newsblur to options

To do

  • test if it really works
  • add newsblur svg icon

Copy link
Owner

@shgysk8zer0 shgysk8zer0 left a comment

Choose a reason for hiding this comment

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

Do you have an account to verify that it works? I'd never heard of NewsBlur before this.

I found an SVG version of the NewsBlur logo at https://www.newsblur.com/media/img/logo.svg but I don't think it can be used via <use> due to gradients and clip path, so it might require using an <img> instead.

@lioman
Copy link
Author

lioman commented Sep 10, 2018

@shgysk8zer0 I have tested it with newsblur and it work if I set the url hard in the calling function. But it isn't saved and I'm not sure how to debug this.

@shgysk8zer0
Copy link
Owner

It should be creating a URL like https://newsblur.com/?url=...

Is that the correct URL and query?
If you console.log that, how is it different from what you're setting?

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.

None yet

4 participants