search

cncf / landscapeapp

🌄Upstream landscape generation application
https://landscapes.dev/
Apache License 2.0
255 stars 125 forks source link

Re-write `mobile.twitter.com` to `twitter.com` #893

Closed jmertic closed 8 months ago

jmertic commented 8 months ago

@AndreyKozlov1984 - can you take a peek at this one? This fixes an issue where the Crunchbase profile is locked and the Twitter starts with mobile.twitter.com; seems safe to rewrite vs reject.

netlify[bot] commented 8 months ago

Deploy Preview for landscapeapp failed.

Name Link
Latest commit b5afaa2abfc2635ecd05f4a9553a0c43dd62204b
Latest deploy log https://app.netlify.com/sites/landscapeapp/deploys/65eb11c5251f2b0007565743
AndreyKozlov1984 commented 8 months ago

Let me just replace "mobile.twitter" with "twitter"!

AndreyKozlov1984 commented 8 months ago

there is one minor issue, off by 1, let me merge and fix it!

jmertic commented 8 months ago

Thank you sir!