Skip to content
This repository has been archived by the owner on Nov 28, 2024. It is now read-only.

CNV-41292: Restricted page for test user on nns #103

Merged
merged 1 commit into from
Jun 21, 2024

Conversation

upalatucci
Copy link
Collaborator

@upalatucci upalatucci commented Jun 21, 2024

porting Access Denied component from console

show Access denied on 403

Before

Screenshot 2024-06-21 at 18 59 30

After

Screenshot 2024-06-21 at 19 01 54

@openshift-ci-robot
Copy link
Collaborator

openshift-ci-robot commented Jun 21, 2024

@upalatucci: This pull request references CNV-41292 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the bug to target the "4.17.0" version, but no target version was set.

In response to this:

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

@openshift-ci openshift-ci bot requested review from hstastna and tnisan June 21, 2024 08:12
Copy link
Contributor

openshift-ci bot commented Jun 21, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: upalatucci

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci-robot
Copy link
Collaborator

openshift-ci-robot commented Jun 21, 2024

@upalatucci: This pull request references CNV-41292 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the bug to target the "4.17.0" version, but no target version was set.

In response to this:

porting Access Denied component from console

show Access denied on 403

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

message?: string;
};

const AccessDenied: React.FC<AccessDeniedProps> = ({ message }) => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Would you mind importing FC above like we do in the kubevirt-plugin repo? I think it's important to keep the same conventions in all repos we maintain.

@pcbailey
Copy link
Collaborator

Could you please add before and after screenshots since this PR creates visual changes?

@openshift-ci-robot
Copy link
Collaborator

openshift-ci-robot commented Jun 21, 2024

@upalatucci: This pull request references CNV-41292 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the bug to target the "4.17.0" version, but no target version was set.

In response to this:

porting Access Denied component from console

show Access denied on 403

Before

Screenshot 2024-06-21 at 18 59 30

After

Screenshot 2024-06-21 at 19 01 54

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

@pcbailey
Copy link
Collaborator

/lgtm

@openshift-ci openshift-ci bot added the lgtm label Jun 21, 2024
@openshift-merge-bot openshift-merge-bot bot merged commit e8e378e into nmstate:main Jun 21, 2024
3 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants