Opened 5 years ago

Closed 5 years ago

#16076 closed New feature (fixed)

Updated locale formats for hungarian language

Reported by: slink Owned by: nobody
Component: Internationalization Version: master
Severity: Normal Keywords: locale, formats
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

The formats module for the hungarian language was very poor, so I updated it.

Attachments (1)

formats.patch​ (1.4 KB) - added by slink 5 years ago.
Patch for hungarian locale formats module

Download all attachments as: .zip

Change History (7)

comment:1 Changed 5 years ago by anonymous

Needs documentation: unset
Needs tests: unset
Patch needs improvement: unset
Triage Stage: Unreviewed → Accepted

comment:2 Changed 5 years ago by Julien Phalip

Needs tests: set
UI/UX: unset

Thanks for the report. Could you provide some tests to go along with your fix?

comment:3 Changed 5 years ago by Russell Keith-Magee

Easy pickings: unset
Needs tests: unset
Triage Stage: Accepted → Ready for checkin

@julien Formats don't require tests; all you'd be testing is that the format string works as expected. It would be like adding tests for translations. We have tests to validate that the format keys work, but we don't need to test that the definitions for each language work as expected.

comment:4 Changed 5 years ago by Jannis Leidel

Is the first entry to TIME_INPUT_FORMATS supposed to have a comma?

Changed 5 years ago by slink

Attachment: formats.patch​ added

Patch for hungarian locale formats module

comment:5 Changed 5 years ago by slink

@jezdez it was mispelled, thanks for the report. Just uploaded the fixed patch.

comment:6 Changed 5 years ago by Jannis Leidel

Resolution: → fixed
Status: new → closed

In [16429]:

Fixed #16076 -- Updated Hungarian l10n formats. Thanks, slink.

Note: See TracTickets for help on using tickets.
Back to Top