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

Startpage url changes to go with non-explorer verison of CB Assignments #20192

Merged

Conversation

h-kataria
Copy link
Contributor

@@ -26,7 +26,7 @@
:startup: true
- :name: chargeback_assignments
:description: Overview / Chargeback / Assignments
:url: /chargeback_assignments/explorer
:url: /chargeback_assignments/index
Copy link
Member

Choose a reason for hiding this comment

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

You don't need index, just remove the last / and everything after and it should work.

@h-kataria h-kataria force-pushed the cb_assignments_startpage_update branch from 21870b0 to f32ebc5 Compare May 20, 2020 18:35
@h-kataria h-kataria force-pushed the cb_assignments_startpage_update branch from f32ebc5 to d97ed24 Compare May 20, 2020 18:42
@@ -26,7 +26,7 @@
:startup: true
- :name: chargeback_assignments
:description: Overview / Chargeback / Assignments
:url: /chargeback_assignments/explorer
:url: /chargeback_assignment
Copy link
Member

Choose a reason for hiding this comment

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

Wait, why are we going to singular here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

plural was a typo, it should have been a singular.

Copy link
Member

@skateman skateman left a comment

Choose a reason for hiding this comment

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

The Seal of Approval

@miq-bot
Copy link
Member

miq-bot commented May 21, 2020

Checked commit h-kataria@d97ed24 with ruby 2.5.7, rubocop 0.69.0, haml-lint 0.28.0, and yamllint
0 files checked, 0 offenses detected
Everything looks fine. 🏆

@chessbyte chessbyte merged commit 64e4d4c into ManageIQ:master May 22, 2020
@chessbyte chessbyte assigned chessbyte and unassigned skateman May 22, 2020
simaishi pushed a commit to simaishi/manageiq that referenced this pull request May 28, 2020
…page_update

Startpage url changes to go with non-explorer verison of CB Assignments

(cherry picked from commit 64e4d4c)
@simaishi
Copy link
Contributor

Jansa backport details:

$ git log -1
commit b42f8c69a73f4cbc1df279b87b7bb3517791fcbb
Author: Oleg Barenboim <[email protected]>
Date:   Fri May 22 15:25:05 2020 -0400

    Merge pull request #20192 from h-kataria/cb_assignments_startpage_update

    Startpage url changes to go with non-explorer verison of CB Assignments

    (cherry picked from commit 64e4d4c8ecb779d3cdb1f1f186a4d14cf89541a6)

@h-kataria h-kataria deleted the cb_assignments_startpage_update branch December 31, 2020 15:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants