Skip to content

refactor(okms): use okms instead of domain #19070

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: project/secret-management-service-b1
Choose a base branch
from
Open

refactor(okms): use okms instead of domain #19070

wants to merge 1 commit into from
+716 −740

Conversation

rjamet-ovh
Copy link
Contributor

ref: #MANAGER-19656

Description

okms domains refactor:
on the UI

  • replace domain with okms domain (translations)

in the code

  • remove all domain references in the code and replace them by okms
  • harmonization on list wording (secretList, okmsList etc...)

@rjamet-ovh rjamet-ovh requested a review from a team as a code owner September 9, 2025 17:13
@rjamet-ovh rjamet-ovh requested review from mathieumousnier and removed request for a team September 9, 2025 17:13
@github-actions github-actions bot added translation required enabler feature New feature has conflicts Has conflicts to resolve before merging labels Sep 9, 2025
@mathieumousnier mathieumousnier force-pushed the project/secret-management-service-b1 branch from b6730c4 to 18d3ee7 Compare September 10, 2025 09:06
@mathieumousnier mathieumousnier requested a review from a team as a code owner September 10, 2025 09:06
@mathieumousnier mathieumousnier requested review from aottr, MaximeBajeux and oalkabouss and removed request for a team September 10, 2025 09:06
@mathieumousnier mathieumousnier force-pushed the project/secret-management-service-b1 branch from 18d3ee7 to 616ad51 Compare September 10, 2025 09:11
refactor(okms): use okms instead of domain
85286d8 
ref: #MANAGER-19656

Signed-off-by: Romain Jamet <[email protected]>
@github-actions github-actions bot removed the has conflicts Has conflicts to resolve before merging label Sep 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mathieumousnier mathieumousnier Awaiting requested review from mathieumousnier mathieumousnier is a code owner automatically assigned from ovh/dev-manager-enterprise-kms

@aottr aottr Awaiting requested review from aottr aottr was automatically assigned from ovh/dev-manager-gcj-all

@MaximeBajeux MaximeBajeux Awaiting requested review from MaximeBajeux MaximeBajeux was automatically assigned from ovh/dev-manager-gcj-all

@oalkabouss oalkabouss Awaiting requested review from oalkabouss oalkabouss was automatically assigned from ovh/dev-manager-gcj-all

Assignees
No one assigned
Labels
enabler feature New feature translation required
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@rjamet-ovh