Skip to content

GPII-2046: Ensure that settings handlers always write to active profiles#477

Open
javihernandez wants to merge 2 commits intoGPII:masterfrom
javihernandez:GPII-2046
Open

GPII-2046: Ensure that settings handlers always write to active profiles#477
javihernandez wants to merge 2 commits intoGPII:masterfrom
javihernandez:GPII-2046

Conversation

@javihernandez
Copy link
Copy Markdown
Member

See https://issues.gpii.net/browse/GPII-2046.
Work in progress - not ready to be merged.

@gpii-bot
Copy link
Copy Markdown

gpii-bot commented Oct 5, 2016

CI job failed. Please visit http://lists.gpii.net/pipermail/ci/ for more details.

@gpii-bot
Copy link
Copy Markdown

gpii-bot commented Oct 5, 2016

CI job passed.

1 similar comment
@gpii-bot
Copy link
Copy Markdown

CI job passed.

"settingsHandlers": {
"configuration": {
"type": "gpii.orca",
"options": {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is there no corresponding linux pull request to this? What does the settings handler do if the user token is not supplied?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

No, there's no linux side for this. As I mentioned back in the meeting we talked about this thing I removed this constraint in Orca settings handler during the last update but I forgot to remove the universal "bits" of it. I agree that this can be ready to go in as a separate pull request since we still need to make some accommodations in UNIVERSAL before the real issue (GPII-2046) can be addressed. Let me know if you want me to make a new JIRA/pr and get this merged ASAP.

@gtirloni
Copy link
Copy Markdown
Contributor

gtirloni commented Feb 3, 2017

ok to test

@gpii-bot
Copy link
Copy Markdown

gpii-bot commented Feb 3, 2017

CI job passed.

@kaspermarkus
Copy link
Copy Markdown
Member

Will pick up again when a higher priority solution needs this functionality

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants