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

get account data as array #37

Merged
merged 1 commit into from
May 11, 2021

Conversation

blizzz
Copy link
Member

@blizzz blizzz commented May 7, 2021

In #36 I actually missed to add the getProperties() call so we continue to work with the array 🙈

Also fixes use of a hard coded value instead of the property constant.

(also for nextcloud/server#26866)

@blizzz blizzz force-pushed the fix/noid/account-as-property-and-constant branch from 8560676 to de57317 Compare May 7, 2021 21:42
- and use the constant

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
@blizzz blizzz force-pushed the fix/noid/account-as-property-and-constant branch from de57317 to 3e96156 Compare May 7, 2021 22:23
Copy link
Member

@MorrisJobke MorrisJobke left a comment

Choose a reason for hiding this comment

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

Looks good 👍

@MorrisJobke MorrisJobke merged commit eead2a7 into master May 11, 2021
@delete-merged-branch delete-merged-branch bot deleted the fix/noid/account-as-property-and-constant branch May 11, 2021 11:18
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.

3 participants