inveniosoftware / invenio-i18n

Invenio internationalization module.
https://invenio-i18n.readthedocs.io
MIT License
5 stars 33 forks source link

rename `get_locale` selector to `select_locale` #123

Open rekt-hard opened 1 year ago

rekt-hard commented 1 year ago

Note: I would really rename invenio_i18n.selectors.get_locale to select_locale, since this might avoid the confusion.

_Originally posted by @slint in https://github.com/inveniosoftware/invenio-i18n/pull/121#discussion_r1246509685_

Also need to checkout other modules which are using this selector

chriz-uniba commented 4 months ago

Does anything speak against this? @rekt-hard @zzacharo @ntarocco

zzacharo commented 4 months ago

Discussed during InvenioRDM workshop 2024: