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

Bug 1895423: Show custom builder image in edit flows BuilderImage section #7021

Merged

Conversation

divyanshiGupta
Copy link
Contributor

@divyanshiGupta divyanshiGupta commented Oct 28, 2020

Fixes:
https://issues.redhat.com/browse/ODC-5015

Analysis / Root cause:
Application is not editable if a custom builder image is used

Solution Description:
Show a new tile for the custom builder image in builder section of edit flows

Screen shots / Gifs for design review:
Peek 2020-10-29 19-20

Unit test coverage report:

Browser conformance:

  • Chrome
  • Firefox
  • Safari
  • Edge

@openshift-ci-robot openshift-ci-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Oct 28, 2020
@openshift-ci-robot openshift-ci-robot added the component/dev-console Related to dev-console label Oct 28, 2020
@divyanshiGupta divyanshiGupta changed the title [WIP]Show custom builder image in edit flows BuilderImage section Show custom builder image in edit flows BuilderImage section Oct 29, 2020
@openshift-ci-robot openshift-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Oct 29, 2020
@divyanshiGupta
Copy link
Contributor Author

/cc: @rohitkrai03

@divyanshiGupta divyanshiGupta force-pushed the jira-issue#5015 branch 2 times, most recently from 3474f34 to 843ae29 Compare October 29, 2020 19:06
@christianvogt
Copy link
Contributor

@divyanshiGupta git url validation is stuck in an infinite loop.

@divyanshiGupta
Copy link
Contributor Author

@divyanshiGupta git url validation is stuck in an infinite loop.

Couldnt reproduce the issue. Please let me know the steps to reproduce it.

@divyanshiGupta divyanshiGupta force-pushed the jira-issue#5015 branch 4 times, most recently from 14b6e0c to b7c88b3 Compare November 3, 2020 15:33
@christianvogt
Copy link
Contributor

/lgtm
/approve

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Nov 5, 2020
@openshift-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: christianvogt, divyanshiGupta

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 openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Nov 5, 2020
@openshift-merge-robot openshift-merge-robot merged commit ea885f8 into openshift:master Nov 6, 2020
@divyanshiGupta divyanshiGupta changed the title Show custom builder image in edit flows BuilderImage section Bug 1895423: Show custom builder image in edit flows BuilderImage section Nov 6, 2020
@openshift-ci-robot
Copy link
Contributor

@divyanshiGupta: All pull requests linked via external trackers have merged:

Bugzilla bug 1895423 has been moved to the MODIFIED state.

In response to this:

Bug 1895423: Show custom builder image in edit flows BuilderImage section

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 kubernetes/test-infra repository.

@divyanshiGupta
Copy link
Contributor Author

/cherrypick release-4.6

@openshift-cherrypick-robot

@divyanshiGupta: new pull request created: #7147

In response to this:

/cherrypick release-4.6

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 kubernetes/test-infra repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. component/dev-console Related to dev-console lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants