Openfoodnetwork: Remove the ability for a superadmin to change the currency in the UI

Created on 28 Dec 2020  Â·  3Comments  Â·  Source: openfoodfoundation/openfoodnetwork

What we should change and why (this is tech debt)

Proposing to remove the ability to change the currency in the UI, as doing so easily break the instance - if the server is not restarted, see bug #6568.

Context

This came up in the context of this discussion and would close bug #6568

tech debt

Most helpful comment

It's defined in a config file through ofn install. I think this pr is a
good change.

On Wed, 30 Dec 2020, 10:05 Rachel Arnould, notifications@github.com wrote:

@openfoodfoundation/core-devs
https://github.com/orgs/openfoodfoundation/teams/core-devs if we remove
it, how is a currency assigned to an instance?

—
You are receiving this because you are on a team that was mentioned.
Reply to this email directly, view it on GitHub
https://github.com/openfoodfoundation/openfoodnetwork/issues/6569#issuecomment-752400155,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMQPOSUL2RW7XCFRRS3EH3SXL3P3ANCNFSM4VL6AVXA
.

All 3 comments

I would like to work on this

@openfoodfoundation/core-devs if we remove it, how is a currency assigned to an instance?

It's defined in a config file through ofn install. I think this pr is a
good change.

On Wed, 30 Dec 2020, 10:05 Rachel Arnould, notifications@github.com wrote:

@openfoodfoundation/core-devs
https://github.com/orgs/openfoodfoundation/teams/core-devs if we remove
it, how is a currency assigned to an instance?

—
You are receiving this because you are on a team that was mentioned.
Reply to this email directly, view it on GitHub
https://github.com/openfoodfoundation/openfoodnetwork/issues/6569#issuecomment-752400155,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMQPOSUL2RW7XCFRRS3EH3SXL3P3ANCNFSM4VL6AVXA
.

Was this page helpful?
0 / 5 - 0 ratings