Skip to content

Conversation

@DenysBurduzhan
Copy link
Contributor

No description provided.

@DenysBurduzhan DenysBurduzhan linked an issue Nov 24, 2024 that may be closed by this pull request
@codacy-production
Copy link
Contributor

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
-0.38% 0.00%
Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (1edd6ce) 330 29 8.79%
Head commit (87f80f3) 345 (+15) 29 (+0) 8.41% (-0.38%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#101) 27 0 0.00%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

Codacy stopped sending the deprecated coverage status on June 5th, 2024. Learn more

1 similar comment
@codacy-production
Copy link
Contributor

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
-0.38% 0.00%
Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (1edd6ce) 330 29 8.79%
Head commit (87f80f3) 345 (+15) 29 (+0) 8.41% (-0.38%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#101) 27 0 0.00%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

Codacy stopped sending the deprecated coverage status on June 5th, 2024. Learn more

<version>1</version>
</dependency>

</dependencies>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Нащо це тут? Схоже він ніде не використовуєтся.

@Override
LocalizationService localizationService = new LocalizationService(Locale.getDefault());

@Override
Copy link
Collaborator

Choose a reason for hiding this comment

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

Це не сюди, це треба зробити у класі PhoneBookContext. Також ти встановлюєш локаль з стстеми, а в нас заплановано що локаль вибирає користувач і зберігає у файлі налаштувань. отже локаль треба взяти звідти.

.addSubcommand("command.list", getBean(ListContacts.class))
.addSubcommand("command.user", getBean(SetUser.class))
.addSubcommand("command.locale", getBean(SetLocale.class))
.execute(args);
Copy link
Collaborator

Choose a reason for hiding this comment

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

Це не текст для користвача це назви команд для picocli їх не треба перекладати. Думаю так воно не заведется.

localizationService.setLocale(new Locale(newLocale));
System.out.println(localizationService.getMessage("success.locale.changed").replace("{0}", newLocale));
System.out.println(localizationService.getMessage("locale.current").replace("{0}", newLocale));
return 0;
Copy link
Collaborator

Choose a reason for hiding this comment

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

ОК. хоча програма закінчить роботу після виконання цєї команди. Чи є потреба міняти поточну локаль?

locale.current=Поточна мова: {0}
locale.available=Доступні мови: {0}


Copy link
Collaborator

Choose a reason for hiding this comment

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

ОК . Але це тільки маленький шматочек тут потрібно внести всі тексти для користувача.

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.

[FR] Create and fill the Localization_UA.properties file

3 participants