Skip to content

Conversation

@Soumi150
Copy link
Contributor

Fixes #5347

Short description of what this resolves:

Fixes spelling of some languages and sorts them in alphabetical order

Checklist

  • I have read the Contribution & Best practices Guide.
  • My branch is up-to-date with the Upstream development branch.
  • The acceptance, integration, unit tests and linter pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)

Screenshot (15)

@vercel
Copy link

vercel bot commented Oct 26, 2020

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/eventyay/open-event-frontend/8d9g9lp9f
✅ Preview: https://open-event-frontend-git-language.eventyay.now.sh

@codecov
Copy link

codecov bot commented Oct 26, 2020

Codecov Report

Merging #5391 into development will increase coverage by 0.05%.
The diff coverage is n/a.

Impacted file tree graph

@@               Coverage Diff               @@
##           development    #5391      +/-   ##
===============================================
+ Coverage        23.23%   23.29%   +0.05%     
===============================================
  Files              489      489              
  Lines             5122     5122              
  Branches            37       37              
===============================================
+ Hits              1190     1193       +3     
+ Misses            3927     3924       -3     
  Partials             5        5              
Impacted Files Coverage Δ
app/services/l10n.js 62.50% <ø> (ø)
app/models/event.js 50.00% <0.00%> (+18.75%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 54059ca...a52806a. Read the comment docs.

@iamareebjamal iamareebjamal changed the title fix spelling and sort languages in alphabetical order fix: spelling and sort languages in alphabetical order Oct 26, 2020
@auto-label auto-label bot added the fix label Oct 26, 2020
'zh_Hans' : '中文 (简化版)',
'zh_Hant' : '中文 (傳統的)',
'zh_Hans' : '中文(简体)',
'zh_Hant' : '中文(繁體)',
Copy link
Member

@iamareebjamal iamareebjamal Oct 27, 2020

Choose a reason for hiding this comment

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

This is present last in Wikipedia. Which sorting have you used?

'id' : 'Bahasa Indonesia',
'ja' : '日本語',
'ru' : 'русский'
'ko' : '한국어',
Copy link
Member

Choose a reason for hiding this comment

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

Korean is last in Wikipedia list as well

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Korean is not last in wikipedia ,its after Francais and Galego.

@Soumi150
Copy link
Contributor Author

Soumi150 commented Oct 27, 2020

I have sorted in alphabetical order .For example Bengali, Chinese, Deutsch and so on. If you wish I would sort it in the wikipedia way.

@iamareebjamal
Copy link
Member

Yes, that's mentioned in the issue comment

'th' : 'ไทย',
'vi' : 'Tiếng Việt',
'fr' : 'Français',
'ko' : '한국어',
Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry, I was referring to another wikipedia page, changed it now.

@iamareebjamal iamareebjamal merged commit 7f54ef6 into fossasia:development Oct 27, 2020
@Soumi150 Soumi150 deleted the language branch October 28, 2020 03:23
sansyrox pushed a commit to sansyrox/open-event-frontend that referenced this pull request Nov 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Footer/Internationalization: Show languages dropdown as written in their original way

2 participants