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

Try to create a dashboard without needed fields #6416

Merged
merged 1 commit into from
Nov 14, 2019

Conversation

jrafanie
Copy link
Member

We previously expected a duplicate dashboard name to raise an error and it no
longer is a problem as of ManageIQ/manageiq#19491

We previously expected a duplicate dashboard name to raise an error and it no
longer is a problem as of ManageIQ/manageiq#19491
@jrafanie
Copy link
Member Author

@miq-bot add_label test

@miq-bot miq-bot added the test label Nov 14, 2019
@jrafanie
Copy link
Member Author

cc @yrudman (thanks for the suggestion on the fix)

@miq-bot
Copy link
Member

miq-bot commented Nov 14, 2019

Checked commit jrafanie@296e287 with ruby 2.5.5, rubocop 0.69.0, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 0 offenses detected
Everything looks fine. ⭐

@himdel
Copy link
Contributor

himdel commented Nov 14, 2019

Merging to get ruby travis green, thanks!
still failing spec:jest fixed separately in #6418

@himdel himdel merged commit 51958e5 into ManageIQ:master Nov 14, 2019
@himdel himdel self-assigned this Nov 14, 2019
@himdel himdel added this to the Sprint 125 Ending Nov 25, 2019 milestone Nov 14, 2019
@jrafanie jrafanie deleted the use_a_new_error branch November 14, 2019 20:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants