Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Query application form server #387

Merged
merged 1 commit into from
May 22, 2020
Merged

Query application form server #387

merged 1 commit into from
May 22, 2020

Conversation

tacruc
Copy link
Collaborator

@tacruc tacruc commented May 22, 2020

Fixes #384
Signed-off-by: Arne Hamann kontakt+github@arne.email

@tacruc
Query application form server
e6b3b36 
Signed-off-by: Arne Hamann <kontakt+github@arne.email>
@tacruc tacruc requested review from julien-nc and jancborchardt May 22, 2020 11:35
jancborchardt
jancborchardt approved these changes May 22, 2020
View reviewed changes
Copy link
Member

@jancborchardt jancborchardt left a comment

Choose a reason for hiding this comment

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

Nice, fixes the issue and keeps the log clean. :) 👍

@jancborchardt jancborchardt merged commit 777b70f into master May 22, 2020
@jancborchardt jancborchardt deleted the query-application branch May 22, 2020 14:22
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@jancborchardt jancborchardt jancborchardt approved these changes

@julien-nc julien-nc Awaiting requested review from julien-nc

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

RuntimeException: App class OCA\Maps\AppInfo\Application is not setup via query() but directly
2 participants
@tacruc @jancborchardt