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

fixed some missing lang entries #117

Closed
wants to merge 5 commits into from
Closed

Conversation

ivoba
Copy link
Contributor

@ivoba ivoba commented Jan 25, 2018

On a fresh install of OXID6 with flow theme i noticed this error:

ERROR: Translation for COOKIE_NOTE_CLOSE not found!

In the shop there is only CLOSE so this ident is missing.

Replaced missing ADD_RECOMM_ADDRECOMMLINK2 with CLICK_HERE as in azure theme.

Further missing SHOP_THEME_sBackgroundPosVertical_left was added.

@CLAassistant
Copy link

CLAassistant commented Jan 25, 2018

CLA assistant check
All committers have signed the CLA.

@ivoba ivoba changed the title added COOKIE_NOTE_CLOSE to lang files fixed some missing lang entries Jan 31, 2018
@ivoba
Copy link
Contributor Author

ivoba commented Mar 9, 2018

@Sieg can you have look here, should be an easy pick

@Sieg Sieg self-assigned this Mar 21, 2018
@@ -157,6 +157,7 @@
'SHOP_THEME_sBackgroundPosHorizontal_right' => 'rechts',
'SHOP_THEME_sBackgroundPosHorizontal_center' => 'zentriert',
'SHOP_THEME_sBackgroundPosVertical' => 'Vertikale Position des Hintergrundbildes',
'SHOP_THEME_sBackgroundPosVertical_left' => 'links',
Copy link
Member

Choose a reason for hiding this comment

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

Vertical left? Something is wrong here :)

Sieg pushed a commit that referenced this pull request Mar 21, 2018
Sieg pushed a commit that referenced this pull request Mar 21, 2018
Use CLICK_HERE in place of currently used missing one.

Related #117
Sieg added a commit that referenced this pull request Mar 21, 2018
@Sieg
Copy link
Member

Sieg commented Mar 21, 2018

Hello @ivoba, I have squashed your commits to be more grouped, and added a line to changelog. I have skipped the last commit regarding the SHOP_THEME_sBackgroundPosVertical_left, as i cannot see any problems regarding this value. We have Horizontal_left key there, anything i have missed? Please write more description how to reproduce the problem if there is one.

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

Successfully merging this pull request may close these issues.

3 participants