-
Notifications
You must be signed in to change notification settings - Fork 136
Feature/share single note #1146
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
Conversation
67f74ce
to
5441c0f
Compare
composer.json
Outdated
@@ -1,6 +1,6 @@ | |||
{ | |||
"require-dev": { | |||
"nextcloud/ocp": "dev-stable25@dev", | |||
"nextcloud/ocp": "dev-master", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you revert this specific change, currently we still cover multiple nextcloud versions and want to run the psalm checks against the ocp package towards the lowest supported.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Class OCP\Share\Events\BeforeShareCreatedEvent
does not exist on "nextcloud/ocp": "dev-stable25@dev",
. We are listening to this event here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right, it is only available since 28.
Let me check if there is a fallback we can use for that then
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I pushed a fallback and added the event to the stub for the static analysis tests.
@luka-nextcloud I just noticed that my previously added commit was overwritten with yours and repushed that. Quick hint, if you use git push --force-with-lease
instead of just a force push you will get notified about changes that have been made in the meantime. https://git-scm.com/docs/git-push#Documentation/git-push.txt---no-force-with-lease
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, thanks for your tip.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Small comments, but otherwise this looks quite good 👍
cc87da7
to
04ecd75
Compare
3d3ab31
to
dd47097
Compare
Signed-off-by: Luka Trovic <[email protected]>
Signed-off-by: Julius Härtl <[email protected]>
Signed-off-by: Julius Härtl <[email protected]>
dd47097
to
72cf34b
Compare
Resolves: #38