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

{liblouis3.6.0 brailletables} #8446

Merged
merged 13 commits into from Jul 17, 2018
Merged

{liblouis3.6.0 brailletables} #8446

merged 13 commits into from Jul 17, 2018

Conversation

@zstanecic
Copy link
Contributor

@zstanecic zstanecic commented Jun 26, 2018

Link to issue number:

closes #8437

Summary of the issue:

In the liblouis 3.6.0, there are some new braille tables for some languages, and others, which were not added in the past.

Description of how this pull request fixes the issue:

This pull request adds new braille tables mentioned in the issue above, and corrects the russian braille table to the proper eight'dot table.
the table for russian whic was used in the past is not a proper russian eight'dot standardised table.
The table in the past was a representation for printing braille.
This pull request adds the following tables:

  1. central Kurdish
  2. czech,
  3. esperanto grade 1,
  4. hungarian grade 2
  5. corrects russian braille table.

Testing performed:

latest master snapshot from source, on the first review, unit test passed,

Known issues with pull request:

none, can be merged to master directly

Change log entry:

=== new features ===
• added czech eight dots, central kurdish, esperanto and hungarian braille tables
=== changes===
• updated the path to the correct russian eight-dots table

zstanecic added 6 commits Jun 26, 2018
note, if the unit tests pass, this can be merged into master
@josephsl
Copy link
Collaborator

@josephsl josephsl commented Jun 26, 2018

@zstanecic
Copy link
Contributor Author

@zstanecic zstanecic commented Jun 26, 2018

@josephsl
Copy link
Collaborator

@josephsl josephsl commented Jun 26, 2018

@zstanecic
Copy link
Contributor Author

@zstanecic zstanecic commented Jun 26, 2018

michaelDCurran added a commit that referenced this pull request Jun 27, 2018
@zstanecic
Copy link
Contributor Author

@zstanecic zstanecic commented Jul 6, 2018

edited the change files for this PR, who can re-review it again?
@leonardder,
@dkager,
@michaelDCurran
@josephsl

…be merged straightforward, as the only master branch is updated.
Copy link
Collaborator

@josephsl josephsl left a comment

If this is the same as the one approved weeks ago, then yes, I motion this be included after a second look by NV Access people. Thanks.

@michaelDCurran michaelDCurran dismissed stale reviews from josephsl and themself via 7fe457b Jul 17, 2018
@michaelDCurran michaelDCurran merged commit 83654e8 into nvaccess:master Jul 17, 2018
1 check passed
1 check passed
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
@nvaccessAuto nvaccessAuto added this to the 2018.3 milestone Jul 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

4 participants