-
Notifications
You must be signed in to change notification settings - Fork 3
ZKUI-437 // Disable/Warn lifecycle expiration workflow for Veeam buckets #862
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
Conversation
Hello hervedombya,My role is to assist you with the merge of this Available options
Available commands
Status report is not available. |
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
Peer approvals must include at least 1 approval from the following list: |
icon={<Icon name="Exclamation-circle" color="statusWarning" />} | ||
variant="warning" | ||
> | ||
This bucket is tagged as being used in a ISV use-case. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Cuervino I think ISV is not the public word we use, better use Connector here no, wdyt ?
This bucket is tagged as being used in a ISV use-case. | |
This bucket is tagged as being used in a connector use-case. |
|
||
useMemo(() => { | ||
if (isISVBucket) { | ||
setValue(`${prefix}understandISVRisk`, false); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think there is an option for revalidating instead of calling trigger()
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
Peer approvals must include at least 1 approval from the following list: |
…ging for connector use-cases
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
Peer approvals must include at least 1 approval from the following list: |
/approve |
I have successfully merged the changeset of this pull request
The following branches have NOT changed:
Please check the status of the associated issue ZKUI-437. Goodbye hervedombya. The following options are set: approve |
No description provided.