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

Remove v1/v2 vestiges in ui #3224

Merged
merged 5 commits into from
Mar 28, 2020
Merged

Remove v1/v2 vestiges in ui #3224

merged 5 commits into from
Mar 28, 2020

Conversation

msorens
Copy link
Contributor

@msorens msorens commented Mar 26, 2020

🔩 Description: What code changed, and why?

Follow-up to #2722, removes the last vestiges of v1 and v2 in the UI.
Review by following the commits for specifics.

⛓️ Related Resources

#2722

👍 Definition of Done

No more v1 and v2 in the UI. Really.

👟 How to Build and Test the Change

No changes in functionality.
Also no changes in tests really; just one test deleted.
So the only way to test the change is to grep v1 and v2... and then sort through the barrage of output.

✅ Checklist

@msorens msorens added automate-auth chore auth-team anything that needs to be on the auth team board labels Mar 26, 2020
@msorens msorens added this to the Auth: Sprint 11 milestone Mar 26, 2020
@msorens msorens requested a review from a team March 26, 2020 23:35
@msorens msorens self-assigned this Mar 26, 2020
Copy link

@blakestier blakestier left a comment

Choose a reason for hiding this comment

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

I know fixing that test is out of scope-- but modifying a comment that doesn't make sense also feels funny. I think if it's too much to just fix it, we should remove that comment/test and make a ticket to deal with it later.

Signed-off-by: michael sorens <msorens@chef.io>
Signed-off-by: michael sorens <msorens@chef.io>
It was returning a false positive due to missing a `detectChanges`.

Signed-off-by: michael sorens <msorens@chef.io>
Signed-off-by: michael sorens <msorens@chef.io>
Signed-off-by: michael sorens <msorens@chef.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auth-team anything that needs to be on the auth team board automate-auth chore emergent
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants