Skip to content
This repository has been archived by the owner on Jul 21, 2023. It is now read-only.

[GAPRINDASHVILI] Add Advanced Options Step to Plan Wizard #518

Merged
merged 1 commit into from Jul 30, 2018

Conversation

michaelkro
Copy link
Contributor

@michaelkro michaelkro commented Jul 26, 2018

@AparnaKarve AparnaKarve changed the title Merge pull request #413 from michaelkro/playbooks [GAPRINDASHVILI] Merge pull request #413 from michaelkro/playbooks Jul 26, 2018
@AllenBW AllenBW added this to In progress in v2v UI via automation Jul 26, 2018
@AllenBW
Copy link
Member

AllenBW commented Jul 26, 2018

Well crap, do we yank out the yup stuff or also backport #453 ?

@michaelkro michaelkro force-pushed the plan-wizard-playbooks branch 3 times, most recently from 19c329e to 3b22cc3 Compare July 26, 2018 17:31
@AparnaKarve
Copy link
Contributor

No schema validators to be included in G, so it needs to be yanked out.

@michaelkro michaelkro changed the title [GAPRINDASHVILI] Merge pull request #413 from michaelkro/playbooks [GAPRINDASHVILI] Add Advanced Options Step to Plan Wizard Jul 27, 2018
Copy link
Member

@AllenBW AllenBW left a comment

Choose a reason for hiding this comment

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

I jumped back into this... looks goooooood!

@simaishi simaishi merged commit 87675ab into ManageIQ:gaprindashvili Jul 30, 2018
v2v UI automation moved this from In progress to Done Jul 30, 2018
@AparnaKarve
Copy link
Contributor

Includes changes done in #472 -- hence no need to backport #472

@michaelkro michaelkro deleted the plan-wizard-playbooks branch August 2, 2018 12:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
v2v UI
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

4 participants