-
Notifications
You must be signed in to change notification settings - Fork 180
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
Refactor extensions to services. #3980
Conversation
Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes. |
82c9a64
to
fdae5f4
Compare
9adb72d
to
6884e22
Compare
💥 Acceptance test Core-API-Tests-ocis-storage-8 failed. Further test are cancelled... |
198b7eb
to
8f74066
Compare
c3289cd
to
7d1c8fa
Compare
c7fad7b
to
97a16b8
Compare
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.
LGTM, there may be still some "extensions" references in the docs. We can check that after the merge.
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.
LGTM
@dragonchaser @wkloucek is it OK to merge this? Or does it need to be merged at some specific time to avoid creating "havoc" for other developers and releases? |
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.
Let us discuss the perfect time please
@phil-davis && @micbar we suggest rebasing & merging this tomorrow afternoon (around 3PM CET) to avoid greater chaos 💥 |
13697f2
to
fe8cb48
Compare
do not merge before .beta4 is released! |
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.
merge only after beta.4 release
Signed-off-by: Christian Richter <crichter@owncloud.com>
Signed-off-by: Christian Richter <crichter@owncloud.com>
Signed-off-by: Christian Richter <crichter@owncloud.com>
Signed-off-by: Christian Richter <crichter@owncloud.com>
Signed-off-by: Christian Richter <crichter@owncloud.com>
fe8cb48
to
0ebfb89
Compare
Kudos, SonarCloud Quality Gate passed! 0 Bugs |
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.
Let's merge it now that beta.4 was released
We have decided to name all extensions, we maintain and provide with ocis,
services from here on to avoid confusion between external extensions and code
we provide and maintain.
fixes: #3926
/cc @mmattel