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

[stable13] ensure userids are strings #252

Merged
merged 1 commit into from Mar 2, 2018

Conversation

Projects
None yet
2 participants
@blizzz
Member

blizzz commented Mar 1, 2018

No description provided.

ensure userids are strings
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>

@blizzz blizzz added this to the Nextcloud 13.0.1 milestone Mar 1, 2018

@blizzz blizzz requested review from rullzer and nickvergessen Mar 1, 2018

@blizzz

This comment has been minimized.

Member

blizzz commented Mar 2, 2018

failing test unrelated to this PR

@nickvergessen

This comment has been minimized.

Member

nickvergessen commented Mar 2, 2018

Seems like

$filteredEmailUsers[$user] ?? false,

is nor working on 5.6, yeah

@nickvergessen nickvergessen merged commit bfe1970 into stable13 Mar 2, 2018

2 of 3 checks passed

continuous-integration/travis-ci/pr The Travis CI build could not complete due to an error
Details
continuous-integration/drone/pr the build was successful
Details
continuous-integration/drone/push the build was successful
Details

@nickvergessen nickvergessen deleted the stable13-247 branch Mar 2, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment