Skip to content
This repository has been archived by the owner on Feb 25, 2022. It is now read-only.

Refactor Settings class #3319

Open
tracywalker opened this issue Jun 21, 2019 · 0 comments
Open

Refactor Settings class #3319

tracywalker opened this issue Jun 21, 2019 · 0 comments
Labels
ARCHIVED CLOSED at time of archiving enhancement New feature request or feature change Test: helper Issue related to test helper functions

Comments

@tracywalker
Copy link
Collaborator

tracywalker commented Jun 21, 2019 •
edited
Loading

moved from Iris2

We have the old design pattern where we create a single _Settings object and then rename it Settings. I'm not sure why we still do this, and it seems that we should just rewrite Settings as a proper class with static properties.
@tracywalker tracywalker added the enhancement New feature request or feature change label Jun 21, 2019
@tracywalker tracywalker mentioned this issue Jun 21, 2019
Closed
@tracywalker tracywalker added the Test: helper Issue related to test helper functions label Jul 17, 2019
@cknowles-admin cknowles-admin added the ARCHIVED CLOSED at time of archiving label Feb 24, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
ARCHIVED CLOSED at time of archiving enhancement New feature request or feature change Test: helper Issue related to test helper functions
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@tracywalker @mwobensmith @cknowles-admin