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

Add support for xlsx files #2164

Merged
merged 14 commits into from
Aug 10, 2018
Merged

Add support for xlsx files #2164

merged 14 commits into from
Aug 10, 2018

Conversation

nijel
Copy link
Member

@nijel nijel commented Aug 9, 2018

Bases on #1888

Before submitting pull request, please ensure that:

  • Every commit has message which describes it
  • Every commit is needed on it's own, if you have just minor fixes to previous commits, you can squash them
  • Any code changes come with testcase
  • Any new functionality is covered by user documentation

mlaggner and others added 13 commits March 7, 2018 11:47
This feature provides an external format (Excel workbook) for download
and upload. The upload is solved by converting the Excel workbook into
a csv file which is used for translation merging

addresses #1879

Signed-off-by: Manuel Laggner <manuel.laggner@gmail.com>
Signed-off-by: Manuel Laggner <manuel.laggner@gmail.com>
Signed-off-by: Manuel Laggner <manuel.laggner@gmail.com>
Signed-off-by: Manuel Laggner <manuel.laggner@gmail.com>
Signed-off-by: Manuel Laggner <manuel.laggner@gmail.com>
also catched the BadZipFile exception which may be thrown by openpyxl

Signed-off-by: Manuel Laggner <manuel.laggner@gmail.com>
Signed-off-by: Manuel Laggner <manuel.laggner@gmail.com>
Signed-off-by: Manuel Laggner <manuel.laggner@gmail.com>

Conflicts:
  docs/admin/install.rst
  weblate/templates/translation-download.html
Signed-off-by: Manuel Laggner <manuel.laggner@gmail.com>
See #1888

Signed-off-by: Michal Čihař <michal@cihar.com>
Signed-off-by: Michal Čihař <michal@cihar.com>
See #1888

Signed-off-by: Michal Čihař <michal@cihar.com>
@nijel nijel self-assigned this Aug 9, 2018
@nijel nijel added this to the 3.2 milestone Aug 9, 2018
It is now basically just a wrapper around CSV and all operations are
performed on csvfile, just the save/load operations are wrapped.

See #1888

Signed-off-by: Michal Čihař <michal@cihar.com>
@codecov
Copy link

codecov bot commented Aug 10, 2018

Codecov Report

Merging #2164 into master will increase coverage by <.01%.
The diff coverage is 95.74%.

@@            Coverage Diff             @@
##           master    #2164      +/-   ##
==========================================
+ Coverage   91.64%   91.65%   +<.01%     
==========================================
  Files         370      372       +2     
  Lines       25140    25226      +86     
  Branches     2802     2813      +11     
==========================================
+ Hits        23040    23121      +81     
- Misses       1329     1333       +4     
- Partials      771      772       +1

@WeblateOrg WeblateOrg deleted a comment Aug 10, 2018
@WeblateOrg WeblateOrg deleted a comment Aug 10, 2018
@WeblateOrg WeblateOrg deleted a comment Aug 10, 2018
@nijel nijel changed the title WIP: Add support for xlsx files Add support for xlsx files Aug 10, 2018
@nijel nijel merged commit 59cce11 into WeblateOrg:master Aug 10, 2018
@nijel nijel deleted the xls branch August 10, 2018 11:06
@nijel nijel mentioned this pull request Aug 10, 2018
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants