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

Fix typo in Observer.php #2899

Closed
wants to merge 1 commit into from
Closed

Conversation

eltociear
Copy link
Contributor

Description (*)

occured -> occurred

Contribution checklist (*)

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All automated tests passed successfully (all builds are green)
  • Add yourself to contributors list

occured -> occurred
@github-actions github-actions bot added the Component: Core Relates to Mage_Core label Jan 5, 2023
@sreichel sreichel marked this pull request as draft January 5, 2023 05:25
@sreichel sreichel added the hold label Jan 5, 2023
@sreichel
Copy link
Contributor

sreichel commented Jan 5, 2023

Set to draft .... see OpenMage/rfcs#8

@addison74
Copy link
Contributor

Based on the feedback here OpenMage/rfcs#8, in such situations the change must be made only on the second column of the CSV translation file. As a programmer, I don't like to see spelling or expression errors in the source code or different values in a CSV row, but that's what the majority wants.

@eltociear - please update you PR by changing only this file https://github.com/OpenMage/magento-lts/blob/4d1869e5985305a1aa8ca5b9627df7f13a0fd464/app/locale/en_US/Mage_Adminhtml.csv in line 107, column 2.

@sreichel sreichel mentioned this pull request Jan 15, 2023
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Core Relates to Mage_Core hold
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants