Nicer serialization of configuration options
ClosedPublic

Authored by ivan on Aug 21 2016, 11:59 AM.

Details

Summary

The previous approach of having an array of object that
hold information about the current config group and the config keys
and values was not really scalable nor intuitive.

Now, the config is an object that has the group names as keys,
and each group has its own configuration (key, value) pairs.

The config groups are identified as a unix-like path

Diff Detail

Repository
R120 Plasma Workspace
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
ivan updated this revision to Diff 6114.Aug 21 2016, 11:59 AM
ivan retitled this revision from to Nicer serialization of configuration options.
ivan updated this object.
ivan edited the test plan for this revision. (Show Details)
ivan added reviewers: Plasma, mart.
Restricted Application added a project: Plasma. · View Herald TranscriptAug 21 2016, 11:59 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
mart edited edge metadata.Aug 22 2016, 8:54 AM

did you try both dump and restore?

ivan added a comment.Aug 22 2016, 12:46 PM
In D2516#47221, @mart wrote:

did you try both dump and restore?

Yes, I've generated the dump and saved it as a layout.js - it loaded as it should.

mart accepted this revision.Aug 22 2016, 12:49 PM
mart edited edge metadata.
This revision is now accepted and ready to land.Aug 22 2016, 12:49 PM
This revision was automatically updated to reflect the committed changes.