Skip to content

Create reusable edit-environment-variables component #1821

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

Merged

Conversation

spadgett
Copy link
Member

@spadgett spadgett commented Jul 11, 2017

This would replace #1815 and #1803

cc @jwforres

@spadgett spadgett force-pushed the environment-editor-component branch 2 times, most recently from 92a3782 to f76ba4b Compare July 11, 2017 14:17
@spadgett spadgett changed the title [WIP] Create reusable edit-environment-variables component Create reusable edit-environment-variables component Jul 11, 2017
@spadgett spadgett requested a review from jwforres July 11, 2017 16:01
@spadgett spadgett force-pushed the environment-editor-component branch 2 times, most recently from 885d88b to e8cad32 Compare July 11, 2017 16:06
Copy link
Member

@jwforres jwforres left a comment

Choose a reason for hiding this comment

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

LGTM for 3.7

@jwforres
Copy link
Member

may want to let QE know to regression test the env editors since its a lot of changes

@openshift-bot openshift-bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jul 13, 2017
@spadgett spadgett force-pushed the environment-editor-component branch 2 times, most recently from 79bbe31 to 1491e66 Compare July 16, 2017 11:41
@spadgett spadgett removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jul 16, 2017
@spadgett
Copy link
Member Author

[test]

@spadgett
Copy link
Member Author

[merge]

- Add environment variable support to stateful sets
- Remove duplicate code from controllers
- Fix problem where the enter key did not submit the environment form
- Fix editor flicker on save
@spadgett spadgett force-pushed the environment-editor-component branch from 1491e66 to b5b8f50 Compare July 20, 2017 01:23
@openshift-bot
Copy link

Evaluated for origin web console test up to b5b8f50

@spadgett
Copy link
Member Author

Rebased to fix merge conflict (which is why it failed).

[merge]

@openshift-bot
Copy link

Evaluated for origin web console merge up to b5b8f50

@openshift-bot
Copy link

Origin Web Console Test Results: SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_request_origin_web_console/15/) (Base Commit: cfb6f13) (PR Branch Commit: b5b8f50)

@openshift-bot
Copy link

openshift-bot commented Jul 20, 2017

Origin Web Console Merge Results: SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_request_origin_web_console/29/) (Base Commit: 5d48301) (PR Branch Commit: b5b8f50)

@openshift-bot openshift-bot merged commit c9b410c into openshift:master Jul 20, 2017
@spadgett spadgett deleted the environment-editor-component branch July 20, 2017 12:07
@spadgett
Copy link
Member Author

@jwforres created a card for QE: https://trello.com/c/3GxVdfi4

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.

Enter key doesn't save changes on Environment tab
3 participants