Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Made search placeholder localizable #5369

Merged
merged 2 commits into from
Dec 6, 2022
Merged

Conversation

uzmannazari
Copy link
Contributor

Summary

Modification of pages search place holder that can be read from language resource files.

Modification of pages search place holder that can be read from language resource files.
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.

Similar question to the other PR if you are moving this to be pulled from localization we need to update the localization files to include this element.

@uzmannazari
Copy link
Contributor Author

Similar question to the other PR if you are moving this to be pulled from localization we need to update the localization files to include this element.

The element is already available in the language resource file

Copy link
Contributor

@valadas valadas left a comment

Choose a reason for hiding this comment

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

Yep appears to be here

Looks good to me

@valadas
Copy link
Contributor

valadas commented Nov 3, 2022

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mitchelsellers mitchelsellers added this to the 9.11.1 milestone Nov 3, 2022
@valadas
Copy link
Contributor

valadas commented Nov 7, 2022

You just have to run ./build.ps1 and get the installer from the artifacts folder and install a site with it. I am pretty sure those quotes need to be removed or it will just literally show that text.

@valadas
Copy link
Contributor

valadas commented Nov 7, 2022

@dnnsoftware/approvers Please do not merge this until we have at least on test report from someone

@david-poindexter
Copy link
Contributor

The build is failing anyway. I am pretty sure those outer quotes need to be removed.

@valadas
Copy link
Contributor

valadas commented Nov 8, 2022

There are " inside of the " string delimiter so that would even close the string before.

Copy link
Contributor Author

@uzmannazari uzmannazari left a comment

Choose a reason for hiding this comment

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

I cant test personabar but i think this will work fine

@valadas valadas merged commit ef79fdf into dnnsoftware:develop Dec 6, 2022
@valadas valadas changed the title Modify search placeholder Made search placeholder localizable Feb 2, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants