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

FFZ emotes no longer work #4430

Closed
Felanbird opened this issue Mar 4, 2023 · 2 comments · Fixed by #4432
Closed

FFZ emotes no longer work #4430

Felanbird opened this issue Mar 4, 2023 · 2 comments · Fixed by #4432
Labels
bug Something isn't working as intended, or works in a confusing/unintuitive way for the user Feature: Emotes Issues related to Emotes

Comments

@Felanbird
Copy link
Collaborator

FFZ used to send us API responses as follows //cdn.frankerfacez.com/xddddd without the necessary https:, so we manually added the https: so we could correctly display emotes.

As of a few hours ago, FFZ has resolved this, but unfortunately we did not want this resolved, and as a result chatterino now sends requests to https:https://cdn.frankerfacez.com/xddddd and said requests fail.

auto emote = Emote{
EmoteName{},
ImageSet{
Url{QString("https:") + jsonUrls.value("1").toString()},
Url{QString("https:") + jsonUrls.value("2").toString()},
Url{QString("https:") +
jsonUrls.value("4").toString()}},
Tooltip{jsonBadge.value("title").toString()}, Url{}};

Url getEmoteLink(const QJsonObject &urls, const QString &emoteScale)
{
auto emote = urls.value(emoteScale);
if (emote.isUndefined() || emote.isNull())
{
return {""};
}
assert(emote.isString());
return {"https:" + emote.toString()};

@Felanbird Felanbird added bug Something isn't working as intended, or works in a confusing/unintuitive way for the user high priority Something is urgent and needs to be worked on as soon as possible. Feature: Emotes Issues related to Emotes labels Mar 4, 2023
@Mm2PL Mm2PL pinned this issue Mar 4, 2023
@Felanbird
Copy link
Collaborator Author

Fixed in #4432 - when this comment is about 30 minutes old you can get the fix in the latest nightly build, 2.4.2 will probably have to come out tomorrow. 🧘‍♂️

@Felanbird Felanbird added Fixed on Nightly Temporary label to inform the user that the fix is available on the nightly build and removed high priority Something is urgent and needs to be worked on as soon as possible. labels Mar 5, 2023
@Felanbird
Copy link
Collaborator Author

@Felanbird Felanbird removed the Fixed on Nightly Temporary label to inform the user that the fix is available on the nightly build label Mar 6, 2023
@Chatterino Chatterino locked as resolved and limited conversation to collaborators Mar 6, 2023
@Felanbird Felanbird unpinned this issue Mar 31, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working as intended, or works in a confusing/unintuitive way for the user Feature: Emotes Issues related to Emotes
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant