Skip to content

PES-2286: shipping method missing title fix#745

Open
FJiskra wants to merge 1 commit intomainfrom
PES-2286-pre-release_title-missing-fix
Open

PES-2286: shipping method missing title fix#745
FJiskra wants to merge 1 commit intomainfrom
PES-2286-pre-release_title-missing-fix

Conversation

@FJiskra
Copy link
Contributor

@FJiskra FJiskra commented Mar 5, 2025

No description provided.

@github-actions

This comment was marked as spam.

@Karpec Karpec force-pushed the PES-2286-pre-release_title-missing-fix branch from 206f866 to 43cec21 Compare March 7, 2025 09:32
@FJiskra FJiskra force-pushed the PES-2286-pre-release_title-missing-fix branch from 43cec21 to e9a7906 Compare March 10, 2025 11:00
return [];
return [
'title' => [
'title' => __( 'Method title', 'packeta' ),
Copy link
Contributor

Choose a reason for hiding this comment

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

Objeví se to někde? Nechal bych tam původní Packeta nebo bych dal třeba Packeta shipping method

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Nenašel jsem.

Bavíme se přeci popisku formulářového pole, ne? "Packeta" nemá moc smysl. BaseShppingMethod tam má toto. Nepleteš si to s výchozí hodnotou?

Copy link
Contributor

Choose a reason for hiding this comment

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

Nevím, jestli je to popisek nebo hodnota. Nechme to tedy na tvou zodpovědnost, uzavírám.

Copy link
Contributor

Choose a reason for hiding this comment

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

Pokud máme title , který jde do překladové funkce, je potřeba zjistit, kde se to uživateli zobrazí a podle toho zvolit adekvátní název . Pokud se nikde nezobrazuje, proč je potřeba to mít v kódu ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Tak BaseShippingMethod dělal Jan. Může to prosím vysvětlit?

@Karpec Karpec changed the base branch from pre-release to main March 12, 2025 07:32
@Karpec Karpec force-pushed the PES-2286-pre-release_title-missing-fix branch from e9a7906 to 3f6b4d9 Compare March 12, 2025 07:34
@Karpec Karpec force-pushed the PES-2286-pre-release_title-missing-fix branch from 3f6b4d9 to 3d2a2e5 Compare March 12, 2025 07:35
return [];
return [
'title' => [
'title' => __( 'Method title', 'packeta' ),
Copy link
Contributor

Choose a reason for hiding this comment

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

Pokud máme title , který jde do překladové funkce, je potřeba zjistit, kde se to uživateli zobrazí a podle toho zvolit adekvátní název . Pokud se nikde nezobrazuje, proč je potřeba to mít v kódu ?

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.

4 participants