Skip to content
This repository has been archived by the owner on Oct 6, 2019. It is now read-only.

Cookieconsent #100

Closed
wants to merge 6 commits into from
Closed

Conversation

donker
Copy link
Contributor

@donker donker commented Oct 6, 2018

This implements the settings management for cookie consent and groups privace settings on one tab.

dnnsoftware/Dnn.Platform#2361
#99

Bonus: it brings back the "display copyright" setting as well.

@dnfclas
Copy link

dnfclas commented Oct 6, 2018

CLA assistant check
All CLA requirements met.

@@ -28,8 +28,5 @@ namespace Dnn.PersonaBar.SiteSettings.Services.Dto
{
public class UpdateOtherSettingsRequest
Copy link
Contributor

Choose a reason for hiding this comment

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

Should this class be removed completely?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I hesitated. Because we may (within the next few weeks) discover settings that should find their way here. Can we continue with this PR and decide this in the run-up to the 9.3 release?

</data>
<data name="plDisplayCopyright.Text" xml:space="preserve">
<value>Display Copyright</value>
</data>
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we create a needs translation label? Do we need to get this translated prior to v9.3? If so, what languages are required?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

True. IMHO it's not critical but it needs translation.

Copy link
Contributor

Choose a reason for hiding this comment

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

IMHO, translations should be open sourced.

Copy link
Contributor

@mitchelsellers mitchelsellers left a comment

Choose a reason for hiding this comment

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

Just need confirmation on my one comment

@@ -1072,7 +1072,7 @@
<value>Enable Localized Content</value>
</data>
<data name="EnableLocalizedContentHelpText.Text" xml:space="preserve">
<value>Enabling localized content allows you to provide translated module content in addition to displaying translated static text. Once localized contetnt is enabled the default site culture will be permanently set and <strong>cannot be changed</strong>.</value>
<value />
Copy link
Contributor

Choose a reason for hiding this comment

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

It looks like this content was removed inadvertently?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes. This should be reverted.

@mitchelsellers
Copy link
Contributor

Closing, was included as part of #136

bdukes pushed a commit that referenced this pull request Mar 19, 2019
CONTENT-7612: localize dnn-file-upload component.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants