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.10] Privacy Consent wording I agree and No #37181

Merged
merged 2 commits into from
Mar 17, 2022

Conversation

pritam825
Copy link
Contributor

@pritam825 pritam825 commented Mar 2, 2022

Pull Request for Issue #22198 .

Summary of Changes

see #22198

Testing Instructions

Actual result BEFORE applying this Pull Request

Screenshot (2031)

Expected result AFTER applying this Pull Request
'No' is replaced by 'I do not agree'

@Shubhamverma2796
Copy link
Contributor

I have tested this item ✅ successfully on 1ab91f1


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

@richard67
Copy link
Member

@zero-24 Do we really want to change that in 3.10? Or shouldn't that be moved to 4.1? Or maybe even 4.2?

@brianteeman
Copy link
Contributor

Its just a string change and it can be considered a bug fix so no reason (in my mind) for it not to go in a patch release.

As for it going in 3.10 - its a bug fix not a feature so why not?

@richard67 richard67 changed the title [3.10] Privacy Content wording I agree and No [3.10] Privacy Consent wording I agree and No Mar 4, 2022
@Ravi-chandra69
Copy link

I have tested this item ✅ successfully on 1ab91f1


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

@richard67
Copy link
Member

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Mar 4, 2022
@zero-24 zero-24 added this to the Joomla 3.10.7 milestone Mar 17, 2022
@zero-24
Copy link
Contributor

zero-24 commented Mar 17, 2022

Merging here thanks.

@zero-24 zero-24 merged commit 901402d into joomla:3.10-dev Mar 17, 2022
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Mar 17, 2022
nibra pushed a commit to nibra/joomla-cms that referenced this pull request Mar 22, 2022
Kostelano added a commit to JPathRu/localisation that referenced this pull request Mar 26, 2022
joomla/joomla-cms#36969 +
joomla/joomla-cms#36861 +
joomla/joomla-cms#37181 +
joomla/joomla-cms#37334 - (только для de-*)
и другие правки из Joomla 4
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

7 participants