-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
Rebrand #25
Comments
Gonna start on item 1 now, hopefully it's not that hard. |
I would just say remove the links entirely, it looks like most of them are only referencing services we probably won't implement like connect and premiere. Seems like a useless addition when used for troubleshooting help as well, anyone would know to visit the website or repository for information. I'm generally not a fan of linking to an external resource for no discernable reason, and these links seem to fall within that category. Examples: |
@dkanada I did change the links (as part of #23, merged in #32), with the note that they'll probably require more changes. Right now, this is more of a cleanup effort as we're trying to stabilize. I'd imagine it's possible to remove some of the connect stuff, and the other signup link stuff, but we'd need to be sure we're not affecting something else first. |
Don't forget to remove mbadmin.net or what ever it is they ping for release versions. |
@drakus72 it's already pointing to non-existing mb3admin.local :) but it might be harder to completely remove it instead of simply shutting down home phoning. |
Might we want jellyf.in rather than jellyfin.media? Just a possibility. Domain's not taken. |
@virtualdxs I debated grabbing that one too, but didn't really seem worth it - we already have 2. |
Need to remove images loading from GitHub/upstream project: https://github.com/MediaBrowser/Emby.Resources/raw/master/images/devices/firefox.png |
I believe this should now be closed. @nvllsvm if you disagree please reopen. |
The text was updated successfully, but these errors were encountered: