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

[3.x] Typo / Spelling BASIS #31362

Closed
wants to merge 1 commit into from
Closed

Conversation

brianteeman
Copy link
Contributor

Fixes obvious typing/spelling error BASIS should be BASIC

code review

This is a 3.x version of #31361 - it has to be done as separate PR because just merging this one into 4 will not fix everything in 4

@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators PR-staging labels Nov 8, 2020
@richard67
Copy link
Member

I have tested this item ✅ successfully on 68edb4a


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31362.

@richard67
Copy link
Member

@ceford As you have tested the other one for J4, can you test this one here for J3, too? Thanks in advance.

@SharkyKZ
Copy link
Contributor

SharkyKZ commented Nov 8, 2020

Isn't removing strings a B/C break?

@richard67
Copy link
Member

Oh yes.

@richard67
Copy link
Member

I have not tested this item.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31362.

@brianteeman
Copy link
Contributor Author

Oops I forgot

Closing

@brianteeman brianteeman closed this Nov 8, 2020
@brianteeman brianteeman deleted the basis branch November 8, 2020 10:34
@richard67
Copy link
Member

Would it make sense to do this fix for J3, only not deleting the old language strings but marking with a deprecation comment line above them, telling to use the new ones, and adding the new language strings (new constants)?

@brianteeman
Copy link
Contributor Author

Dont think its worth it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Language Change This is for Translators
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants