Skip to content
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

When editing PID profiles from CMS, change the profile immediately #3433

Merged
merged 1 commit into from Jun 25, 2018

Conversation

Projects
None yet
1 participant
@fiam
Copy link
Member

commented Jun 24, 2018

This way we make sure we're always reading and writing to the
selected PID profile without having to rely on so much intermediate
data (with its assocciated RAM costs). It also fixes the problem
when data from a PID profile could accidentaly overwrite another.

Fixes #3064

When editing PID profiles from CMS, change the profile immediately
This way we make sure we're always reading and writing to the
selected PID profile without having to rely on so much intermediate
data (with its assocciated RAM costs). It also fixes the problem
when data from a PID profile could accidentaly overwrite another.

Fixes #3064

@fiam fiam added this to the 2.0 milestone Jun 24, 2018

@fiam fiam merged commit 9905ff8 into development Jun 25, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@fiam fiam deleted the agh_fix_pid_editing_from_cms branch Jun 25, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.