Skip to content

Add support for verified Medium aliases#25

Merged
fferrin merged 2 commits into
fferrin:masterfrom
dhyeyinf:add-verified-medium-domains
Jun 27, 2025
Merged

Add support for verified Medium aliases#25
fferrin merged 2 commits into
fferrin:masterfrom
dhyeyinf:add-verified-medium-domains

Conversation

@dhyeyinf
Copy link
Copy Markdown
Contributor

This PR updates the manifest.json to add support for the following verified Medium-powered domains:

  • https://stories.byburk.net/*
  • https://theinnovationhub.medium.com/*
  • https://neuralnetworkpress.com/*

Each domain was:

  • Confirmed to be hosted on Medium
  • Displays paywalled content
  • Works with Freedium redirection

These were extracted from a previous PR (#24 ) and refined based on your feedback.

Thank you for maintaining this extension!

@fferrin
Copy link
Copy Markdown
Owner

fferrin commented Jun 23, 2025

Hi @dhyeyinf . The second domain can not be found and it should also be included under the first rule ("https://*.medium.com/*"). And the third one does not look as Medium posts. Can you please send me a post from each domain so I can verify that the button does not show up there but it does once I apply your changes?

@dhyeyinf
Copy link
Copy Markdown
Contributor Author

https://stories.byburk.net/i-made-1-million-writing-online-debde99913e4
https://theinnovationhub.medium.com/why-ai-is-reinventing-startups-2e9a354da4df

  • neuralnetworkpress.com was initially included, but upon checking, it's not a Medium-powered domain and shows a custom 404.

Thanks again for reviewing and maintaining this extension!

@fferrin
Copy link
Copy Markdown
Owner

fferrin commented Jun 23, 2025

Thanks for the links. The first one work fine but the second one show a 404. However, if you enter the link you will see that the blue button is present on the page, so the only addition needed from your PR is line 69. If you can make the changes, I'll merge them today

@dhyeyinf
Copy link
Copy Markdown
Contributor Author

Sure, I will make the necessary changes. Thank you.

@dhyeyinf
Copy link
Copy Markdown
Contributor Author

I've made the changes as requested — the PR now only includes the addition of ://stories.byburk.net/* in the matches array (line 69). All other domains have been removed.

Thanks again for your quick review! Let me know if anything else is needed.

@dhyeyinf
Copy link
Copy Markdown
Contributor Author

Hi 👋 just following up on this — I made the requested change and updated the PR to include only the verified domain stories.byburk.net (line 69), as discussed. It’s been a few days, so I just wanted to check if there's anything else needed from my side.

Appreciate your time and effort in maintaining this extension — looking forward to your feedback or merge whenever you get a chance!

@fferrin fferrin self-requested a review June 27, 2025 08:28
Copy link
Copy Markdown
Owner

@fferrin fferrin left a comment

Choose a reason for hiding this comment

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

Thanks for adding this!

@fferrin fferrin merged commit 961c4ea into fferrin:master Jun 27, 2025
2 checks passed
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.

2 participants