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
Feature/2385/config #2401
Feature/2385/config #2401
Conversation
1 similar comment
gitlabRedirectPath: /auth/gitlab.com | ||
gitlabScope: api | ||
|
||
googleScope: profile email |
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.
space separated?
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.
Yes, that's how it is in dockstore.model.ts
import io.swagger.annotations.ApiModel; | ||
import org.apache.commons.beanutils.BeanUtils; | ||
|
||
@ApiModel("Config") |
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.
Seems a bit generic
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.
Agreed, but I'm having a hard time thinking of anything better. Any ideas?
UIConfig? Configuration? UIConfiguration? Those aren't really any better, but I'm drawing a blank.
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 kept the name but added a description.
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.
On fixed build
#2385 Adds a new endpoint, /metadata/config to retrieve information to be used by the UI. As part of this, make githubClientID and githubClientSecret simple strings -- they were List<String>. The List was to support the transition from ui1 to ui2, but now that the UI will soon be retrieving the client id from the web service, it no longer makes sense.
Missed checking in yaml files
Add a description to Config
c776cf5
to
39784cb
Compare
Expose configuration information for consumption by the API.
In doing so, change githubClientId and githubClientSecret to be
String
-- they wereList<String>
for legacy reasons for the migration from ui1 to ui2.Because of that change I needed to update dockstoreTest.yml, which was specifying those values as arrays, and hence the secret archive, even though nothing secret in the secret archive changed.
The Google Tag Manager id is not in here; I will need to think about that one separately.