-
Notifications
You must be signed in to change notification settings - Fork 181
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
Update Dependencies on Ocis Extensions [Settings] #264
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. |
go.mod
Outdated
|
||
// replace github.com/owncloud/ocis-proxy => ../ocis-proxy | ||
|
||
// replace github.com/owncloud/ocis-accounts => ../ocis-accounts | ||
|
||
// replace github.com/owncloud/ocis-settings => ../ocis-settings | ||
|
||
// replace github.com/owncloud/ocis-migration => ../ocis-migration |
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.
Currently it uses relative replace lines. I doubt this is what you intend.
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.
just comments, forgot to get rid of them :L
close as obsolete ? |
For the upcoming release, we have a bunch of open PR that need to be merged. To avoid having local replace statements we're locking
ocis-*
extensions to commits on each PR latest version.Ideally we would use branch names, as in:
go get github.com/owncloud/ocis-settings@feature/whatever
but since many PR use feature branches as naming conventions andgo get
does not support slashes on branch names, we have to stick to the commit hash.This PR should provide with a build-able binary, with the proxy middleware enabled.
To run, OIDC needs to be configured on the proxy config.json:
where
proxy-example.json
contents are thisThis PR serves as a testing context for the settings services. There are things that need to be done for a beta release (listed below). After that, all the branches linked in this PR will have to get merged into their respective masters.
Phoenix & SDK
Accounts Service
me
account uuid to an actual account uuid.Proxy
Settings Service