Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Update account dropdowns #2959

Merged
merged 5 commits into from
Oct 31, 2016
Merged

Update account dropdowns #2959

merged 5 commits into from
Oct 31, 2016

Conversation

jacogr
Copy link
Contributor

@jacogr jacogr commented Oct 29, 2016

@jacogr jacogr added A0-pleasereview 🤓 Pull request needs code review. A2-insubstantial 👶 Pull request requires no code review (e.g., a sub-repository hash update). M5-ui and removed A2-insubstantial 👶 Pull request requires no code review (e.g., a sub-repository hash update). labels Oct 29, 2016
@gavofyork gavofyork added A8-looksgood 🦄 Pull request is reviewed well. A7-looksgoodtestsfail 🤖 Pull request is reviewed well, but cannot be merged due to tests failing. and removed A0-pleasereview 🤓 Pull request needs code review. A8-looksgood 🦄 Pull request is reviewed well. labels Oct 29, 2016
@jacogr
Copy link
Contributor Author

jacogr commented Oct 30, 2016

Sadly, nothing I can do with the test failures at this point in time. On the JS side where the changes was made all tests and checks pass (lint, building, test), but no matter how many times master is merged in, GitLab never has success. Will come back to it when master gets to a build state again.

@gavofyork gavofyork closed this Oct 31, 2016
@gavofyork gavofyork reopened this Oct 31, 2016
@gavofyork gavofyork added A8-looksgood 🦄 Pull request is reviewed well. and removed A7-looksgoodtestsfail 🤖 Pull request is reviewed well, but cannot be merged due to tests failing. labels Oct 31, 2016
@gavofyork gavofyork merged commit d086251 into master Oct 31, 2016
@gavofyork gavofyork deleted the jg-account-dropdown branch October 31, 2016 11:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants