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

Acf certificate upload from GUI is failing for the service/admin users #184

Merged
merged 2 commits into from
Feb 14, 2024

Conversation

Renuka9527
Copy link

@Renuka9527 Renuka9527 commented Feb 9, 2024

Description: Unable to add the Acf certificate from GUI.
Reverted “Service Login Certificate” updated using translation file change from store as we are using the certificates check in many other places which is causing the Acf certificate upload issue.
Added it in the certificates vue with 'ServiceLogin Certificate' check and updating the value from translation file.

Defect: https://jazz07.rchland.ibm.com:13443/jazz/web/projects/CSSD#action=com.ibm.team.workitem.viewWorkItem&id=59687

https://jazz07.rchland.ibm.com:13443/jazz/web/projects/CSSD#action=com.ibm.team.workitem.viewWorkItem&id=59643
Issue: ibm-openbmc/openbmc#305

Screenshot:
image
image

@Renuka9527 Renuka9527 changed the title Reverted “Service Login Certificate” updated using translation file from store Acf certificate upload from GUI is failing for the service/admin users Feb 9, 2024
@Nikhil-Ashoka
Copy link
Collaborator

@Renuka9527, The en-US.json file is not updated. Please pull the code.

Copy link
Collaborator

@Nikhil-Ashoka Nikhil-Ashoka left a comment

Choose a reason for hiding this comment

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

Pull to the latest code and push the changes. It is showing as blank.

@Nikhil-Ashoka Nikhil-Ashoka self-requested a review February 14, 2024 09:50
@rfrandse rfrandse merged commit 5063ec6 into ibm-openbmc:1060 Feb 14, 2024
rfrandse pushed a commit that referenced this pull request Feb 14, 2024
rfrandse pushed a commit that referenced this pull request Feb 14, 2024
rfrandse pushed a commit that referenced this pull request Feb 14, 2024
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.

5 participants