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

add TargetCluster URL in InternalWorkspace's status#304

Merged
openshift-merge-bot[bot] merged 7 commits intokonflux-workspaces:mainfrom
filariow:workspace-url
Oct 9, 2024
Merged

add TargetCluster URL in InternalWorkspace's status#304
openshift-merge-bot[bot] merged 7 commits intokonflux-workspaces:mainfrom
filariow:workspace-url

Conversation

@filariow
Copy link
Member

@filariow filariow commented Sep 27, 2024

requires #308

@filariow
Copy link
Member Author

/hold

sadlerap
sadlerap previously approved these changes Sep 27, 2024
@openshift-ci openshift-ci bot added lgtm and removed lgtm labels Sep 27, 2024
@filariow filariow force-pushed the workspace-url branch 4 times, most recently from 956071e to 41b1c6e Compare October 4, 2024 13:47
This commit adds a test case for checking the behavior of the system
when an user with a long name (>20 characters) onboards

Signed-off-by: Francesco Ilario <filario@redhat.com>
Signed-off-by: Francesco Ilario <filario@redhat.com>
Signed-off-by: Francesco Ilario <filario@redhat.com>
Signed-off-by: Francesco Ilario <filario@redhat.com>
Signed-off-by: Francesco Ilario <filario@redhat.com>
sadlerap
sadlerap previously approved these changes Oct 7, 2024
@openshift-ci openshift-ci bot added the lgtm label Oct 7, 2024
@sadlerap
Copy link
Member

sadlerap commented Oct 7, 2024

/hold

@sadlerap
Copy link
Member

sadlerap commented Oct 7, 2024

With those changes you've suggested, lgtm

Signed-off-by: Francesco Ilario <filario@redhat.com>
@filariow filariow requested a review from sadlerap October 8, 2024 13:03
@openshift-ci openshift-ci bot added the lgtm label Oct 9, 2024
@openshift-ci
Copy link

openshift-ci bot commented Oct 9, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: filariow, sadlerap

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

The pull request process is described here

Details 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

@filariow
Copy link
Member Author

filariow commented Oct 9, 2024

/unhold

@openshift-merge-bot openshift-merge-bot bot merged commit 14676a4 into konflux-workspaces:main Oct 9, 2024
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.

2 participants