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

UHF-8290 getCurrentLanguage parameter #493

Merged
merged 6 commits into from
Apr 12, 2023
Merged

UHF-8290 getCurrentLanguage parameter #493

merged 6 commits into from
Apr 12, 2023

Conversation

rpnykanen
Copy link
Contributor

@rpnykanen rpnykanen commented Apr 6, 2023

AnnouncementBlock.php: using type_content on block build method.

Add new announcement with only finnish language. Add another with only english language. You should see the correct announcement always.

paragraphs_news_list_preprocess: using type_content on preprocess.

Add news list paragraph to content (multiple translations). The urls pointing to front-page instance's news archive should
have correct url + lang code.

LinkedEvents.php: using type_content on requests language parameter.

Add events paragraph to a page. Fill the info (url for example https://tapahtumat.hel.fi/en/events?text=jooga).
The content should be on correct language.

React and share: using type_content on block's build method.

Make sure you have REACT_AND_SHARE_APIKEY_{LANGCODE} set as env variable or otherwise the block won't show up. Check test environment for correct key. Now go to any page and you should see the block rendered with correct language. The language is only used to pass correct api key to the javascript.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Apr 6, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@github-advanced-security
Copy link

You have successfully added a new SonarCloud configuration ``. As part of the setup process, we have scanned this repository and found no existing alerts. In the future, you will see all code scanning alerts on the repository Security tab.

Copy link
Contributor

@Jussiles Jussiles left a comment

Choose a reason for hiding this comment

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

🚀

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

Successfully merging this pull request may close these issues.

2 participants