Skip to content

PRESS11-203: Send the user's preferred language to Hiive and forward it to CTB#41

Merged
circlecube merged 3 commits intomainfrom
PRESS11-203
Apr 14, 2025
Merged

PRESS11-203: Send the user's preferred language to Hiive and forward it to CTB#41
circlecube merged 3 commits intomainfrom
PRESS11-203

Conversation

@girish-lokapure
Copy link
Copy Markdown
Contributor

@girish-lokapure girish-lokapure commented Apr 10, 2025

Proposed changes

Jira: https://jira.newfold.com/browse/PRESS11-203
This change is part of the referenced ticket. It retrieves the WordPress locale, sets it in window.newfoldruntime, and makes it accessible via JavaScript. The region-specific part of the locale is stripped, and the resulting value is sent to Hiive through an iframe.

Hiive then forwards this value to the CBT call. PR : https://github.com/newfold-labs/laravel-hiive/pull/736

Type of Change

Production

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Dependency update
  • Refactoring / housekeeping (changes to files not directly related to functionality)

Development

  • Tests
  • Dependency update
  • Environment update / refactoring
  • Documentation Update

Video

Checklist

  • I have read the CONTRIBUTING doc
  • I have viewed my change in a web-browser
  • Linting and tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)

Further comments

wpscholar
wpscholar previously approved these changes Apr 14, 2025
@circlecube circlecube merged commit ff324fc into main Apr 14, 2025
2 of 3 checks passed
@circlecube circlecube deleted the PRESS11-203 branch April 14, 2025 17:16
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.

3 participants