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

Replace all api calls that retrieve LanguageKey by the a new APIs that return LanguageVariable (Content) #28251

Closed
fabrizzio-dotCMS opened this issue Apr 16, 2024 · 0 comments · Fixed by #28544

Comments

@fabrizzio-dotCMS
Copy link
Contributor

fabrizzio-dotCMS commented Apr 16, 2024

Parent Issue

#25603

User Story

Write a new API Method to retrieve all Language Variables as lightly as possible. No contentlet additional info should be making it into the returned object. Only identifier, key, value, and language.
This way the resulting method can be cached with a smaller memory footprint. Once this method is in place and caching is properly handled we need to identify the actual use of the old APIs and replace them with the respective calls to the new ones. So we only rely on the language variables stored as content.
This method should ignore permissions and should content state (working or live) any LanguageVariable should be returned.

Acceptance Criteria

After completing this task No use of the old LanguageKey API should remain active in code.
The Old APIS should be marked as deprecated. So they can be easily identified shortly for removal.
The new APIs method will make it possible to load and cache all Language Variables despite of state and permissions

Proposed Objective

Core Features

Proposed Priority

Priority 3 - Average

External Links... Slack Conversations, Support Tickets, Figma Designs, etc.

No response

Assumptions & Initiation Needs

No response

Quality Assurance Notes & Workarounds

No response

Sub-Tasks & Estimates

No response

@nollymar nollymar removed the Triage label Apr 17, 2024
@fabrizzio-dotCMS fabrizzio-dotCMS changed the title Replace all api calls to that retrieve LanguageKey by the new APIs that return LanguageVariable (Content) Replace all api calls that retrieve LanguageKey by the new APIs that return LanguageVariable (Content) Apr 23, 2024
@fabrizzio-dotCMS fabrizzio-dotCMS self-assigned this Apr 23, 2024
fabrizzio-dotCMS added a commit that referenced this issue Apr 23, 2024
fabrizzio-dotCMS added a commit that referenced this issue Apr 24, 2024
fabrizzio-dotCMS added a commit that referenced this issue Apr 24, 2024
fabrizzio-dotCMS added a commit that referenced this issue Apr 24, 2024
fabrizzio-dotCMS added a commit that referenced this issue Apr 24, 2024
fabrizzio-dotCMS added a commit that referenced this issue Apr 26, 2024
fabrizzio-dotCMS added a commit that referenced this issue Apr 26, 2024
fabrizzio-dotCMS added a commit that referenced this issue Apr 30, 2024
github-merge-queue bot pushed a commit that referenced this issue May 1, 2024
* #28325 upping quarkus platform

* #28325 2.16.12.Final is the leatest version on which I could compile the project

* #28325

* #28251

* #28325 adjusting our test to run with the new quarkus platf ver

* #28325 doc

* #28325
@fabrizzio-dotCMS fabrizzio-dotCMS changed the title Replace all api calls that retrieve LanguageKey by the new APIs that return LanguageVariable (Content) Replace all api calls that retrieve LanguageKey by the a new APIs that return LanguageVariable (Content) May 1, 2024
fabrizzio-dotCMS added a commit that referenced this issue May 13, 2024
fabrizzio-dotCMS added a commit that referenced this issue May 13, 2024
fabrizzio-dotCMS added a commit that referenced this issue May 13, 2024
fabrizzio-dotCMS added a commit that referenced this issue May 16, 2024
fabrizzio-dotCMS added a commit that referenced this issue May 16, 2024
fabrizzio-dotCMS added a commit that referenced this issue May 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants