Skip to content

feat: coop management - #822

Open
crosspolar wants to merge 8 commits into
masterfrom
coop_management
Open

feat: coop management#822
crosspolar wants to merge 8 commits into
masterfrom
coop_management

Conversation

@crosspolar

Copy link
Copy Markdown
Contributor

I figured, it would be nice if Vorstand is not relying on us to change the address of SuperCoop :). could superseed #819

@Theophile-Madet Theophile-Madet left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I'm not sure it's worth the added complexity for something that changes every 5 years, but if we do want it, could we make it more generic?
Instead of a CoopAddress model we could have something like

class CustomizableString(models.Model):
    key = models.CharField(...)
    value = models.CharField(...)

Where the list of keys is defined as constants, and the keys are coop_name, coop_full_name...
That would let us add more strings by adding a constant and we're not bound to the CoopAddress name so we can put other things in it.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants