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

Facebook Likebox: Return a language when none found. #2834

Merged
merged 1 commit into from
Oct 20, 2015

Conversation

kraftbj
Copy link
Contributor

@kraftbj kraftbj commented Oct 9, 2015

If no language is used in the URL to grab the SDK, it fails. We fallback to en_US when we don't know of a language to try, so let's do the same when a known language as no facebook_locale.

@kraftbj kraftbj added [Type] Bug When a feature is broken and / or not performing as intended [Feature] Extra Sidebar Widgets [Status] Needs Review This PR is ready for review. labels Oct 9, 2015
@kraftbj kraftbj added this to the 3.8 milestone Oct 9, 2015
@dereksmart
Copy link
Member

LGTM

@dereksmart dereksmart added [Status] Ready to Merge Go ahead, you can push that green button! and removed [Status] Needs Review This PR is ready for review. labels Oct 9, 2015
samhotchkiss added a commit that referenced this pull request Oct 20, 2015
Facebook Likebox: Return a language when none found.
@samhotchkiss samhotchkiss merged commit ba44501 into master Oct 20, 2015
@samhotchkiss samhotchkiss removed the [Status] Ready to Merge Go ahead, you can push that green button! label Oct 20, 2015
@samhotchkiss samhotchkiss deleted the fix/fb-widget-lang branch October 20, 2015 20:00
coder-karen added a commit that referenced this pull request Feb 7, 2023
coder-karen added a commit that referenced this pull request Feb 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Extra Sidebar Widgets [Type] Bug When a feature is broken and / or not performing as intended
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants