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

Translation of delivery_time label #124

Closed
DanieliMi opened this issue Nov 8, 2018 · 6 comments
Closed

Translation of delivery_time label #124

DanieliMi opened this issue Nov 8, 2018 · 6 comments
Assignees

Comments

@DanieliMi
Copy link

General

  • Module version: 2.0.1
  • Magento version: 2.2.6
  • PHP version: 7.0.32

Issue description

The translation of the label for delivery time is managed over the dictionary translation as of #74. But isn't it better to stay closer to magento and use the Manage Labels functionality?

The label here needs to be changed to "default" in order to use magentos functionality.

@amenk
Copy link

amenk commented Apr 3, 2019

Not to confuse with #80

@amenk
Copy link

amenk commented Apr 25, 2019

vendor/firegento/magesetup2/view/frontend/templates/product/list/delivery.phtml:25

if ($_attributeLabel && $_attributeLabel == 'default') {
     $_attributeLabel = $_product->getResource()->getAttribute($_code)->getFrontendLabel();
}

I found this code ... it looks like in

vendor/firegento/magesetup2/view/frontend/layout/catalog_delivery_info.xml:14

we just have to use the label "default" and it would work?

Why not just set this default as default?

@codedge
Copy link

codedge commented Sep 23, 2019

This still is an issue in the recent version :(

@itsolon
Copy link

itsolon commented Oct 29, 2019

Is there any update.?
i have Delivery Time at product listing activated, but doing so .. no translate csv is drawn wherever i try to put the text in
i tried, in attribute itself
in i18n folder from vendor firegento already is the text inside works only for prodcut detail page
in i18n from my template as last measure does not work
thats all
i could alter the file delivery
the easiest way is to switch of this functionality
but you worked it in so lets perhaps use it?

I know you seo freaks out there you think now .. if he switches on then the seo strength of my keywors is washed out because of the use of hundreds delivery time haha.
best Regards
Markus

@sprankhub sprankhub self-assigned this Jun 27, 2020
@sprankhub
Copy link
Member

I tend to replace "Delivery Time" here:

<argument name="at_label" xsi:type="string" translate="true">Delivery Time</argument>

with default, so that the label comes from the backend. However, this could mean that many German shops have the word in English on their product page after the next MageSetup update. They would need to manually change the label in the backend. I am fine with it, but I do not want to decide it on my own :) @firegento/magesetup2-maintainers, what do you think?

@frostblogNet
Copy link
Member

I`m fine with that too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants