search

ozwillo / ozwillo-portal

Ozwillo's user portal
https://www.ozwillo.com
GNU Affero General Public License v3.0
1 stars 2 forks source link

required field "geographical area of interest" when publishing an app or a service #242

Closed OzBruno closed 9 years ago

OzBruno commented 9 years ago

When publishing an app or a service in the catalog, the provider must fill in a "geographical area of interest" for which the app or service is relevant. This filed can be multivalued and composed of areas of the datacore's geo model. In case of a public body publishing a service, the "geographical area of interest" should be prepopulated by its jurisdiction but remaining editable.

OzBruno commented 9 years ago

This filed has to be used by the geographical search engine in the catalog.

tbroyer commented 9 years ago

I disagree about the “must”, for a couple reasons:

  1. it's a breaking change. App factories and applications publishing services themselves will break (and in the case of app-factories, that means instantiating the application will fail). Users willing to edit services that are already published will be annoyed as they'll probably be forced to now provide a geographical area.
  2. not all apps/services have a geographical area of interest: an online storage application (à la Google Drive, Dropbox or MS OneDrive), calendar or other so-called "productivity" tools, a generic mapping tool (OzMaps), a wiki, etc.
mdutoo commented 9 years ago

See detailed behaviour in caveat in https://github.com/pole-numerique/oasis-portal/issues/138#issuecomment-134219916

mdutoo commented 9 years ago

Forgotten February UI mock:

apps_mgt_1

OzBruno commented 9 years ago

@tbroyer OK not to render the filed mandatory, especially as it is for the v1. @mdutoo OK to start first with a single value

ilucatero commented 9 years ago

As an alternative solution, I tried to use another framework for the select2 called react-select (https://github.com/JedWatson/react-select), but it still has some issues and the solution requires possible further tests and configurations to make it show the field - and at this point for this solution I couldn't make it work with multi-select as well.