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

ancestorKeysForKey always returns an array #6205

Merged
merged 2 commits into from
Aug 1, 2019
Merged

Conversation

meirish
Copy link
Contributor

@meirish meirish commented Feb 11, 2019

@johncowen pointed out that it'd be nicer to always return an array from this helper so that future uses are less error-prone. This PR changes it to do that and adapts usage of the util accordingly in other places.

Copy link

@johncowen johncowen left a comment

Choose a reason for hiding this comment

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

LGTM! 🙌

@meirish meirish merged commit 0271486 into master Aug 1, 2019
@meirish meirish deleted the ui-util-feedback branch August 1, 2019 20:49
meirish added a commit that referenced this pull request Aug 5, 2019
* ancestorKeysForKey always returns an array (#6205)

* clear namespaces on logout and properly render nested namespaces in the namepace picker (#7186)

* reset namespace cache when a user logs out

* fix issue where if you log in to a namespace with an initial /, nested namespaces would not show up in the navigation

* set an empty list of namespaces instead of ignoring error

* add tests for namespace bugs

* use this consistently in template
@chrishoffman chrishoffman added this to the 1.2.1 milestone Oct 23, 2019
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.

6 participants