Skip to content

Conversation

@ovari
Copy link

@ovari ovari commented Aug 7, 2019

Signed-off-by: Óvári <ovari123@zoho.com>

Locale: hu (magyar - Hungarian)

Licence: AGPL3+ or any other compatible license

Description

Add hu locale

Features

  • hu (magyar - Hungarian) locale added
  • few minor fixes

Screenshots or screencasts

Caveats

  • Don't know how to do git

Tests

Test plan

Tested on

Not required as it just adds the hu locale

  • Windows/Thunderbird
  • Linux/Thunderbird
  • macOS/Thunderbird

TODO

  • None before merging

Check list

  • Code is properly documented
  • Code is properly formatted
  • Commits have been squashed
  • Documentation (manuals or wiki) has been updated or is not required

Reviewers

Guillaume Viguier-Just (@guillaumev)
Olivier Paroz (@oparoz)

@ovari
Copy link
Author

ovari commented Aug 7, 2019

Don't know how to do git on local computer. How can DCO be fixed as we are using the web-based IDE? Thank you

@ovari
Copy link
Author

ovari commented Aug 12, 2019

Guillaume Viguier-Just (@guillaumev)
Olivier Paroz (@oparoz)
Thomas Bille (@tbille)
@uli-heller
@NDev8

Could you please accept this pull request?

Thank you

@gary-kim
Copy link
Member

For your DCO, you can manually add the following to your commit message.

Signed-off-by: Random J Developer <random@developer.example.org>

Although, could you squash your commits by any chance? Thank you.

@ovari
Copy link
Author

ovari commented Sep 30, 2019

Added DCO to #79 (comment).

Sorry, not sure how to squash commits using GitHub Web Editor.

@ovari
Copy link
Author

ovari commented Jan 6, 2020

Could you please merge this so that our fork can be deleted? Thank you

joendres added a commit to joendres/nextcloud-filelink that referenced this pull request Feb 18, 2020
Release 3.4

Closes nextcloud#70, nextcloud#71, and nextcloud#79

See merge request joendres/filelink-nextcloud!47
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