DigitalCommons / mykomap

A web application for mapping initiatives in the Solidarity Economy
3 stars 0 forks source link

[Workers.coop] Conduct a Filter on map opening #205

Closed ColmDC closed 4 months ago

ColmDC commented 1 year ago

Please track against the clockify project [Workers.coop]

Description

Expand MM configuration options to support the specification of a text search and or filter combination for when a MM opens.

Acceptance Criteria

It is possible to specify the search/ filter combination within the configuration file or via URL parameters for the map. We can demonstrate the workers.coop map opening with only Active co-ops displayed but the filter can be changed to display all co-ops or the other status'. We can demonstrate the dotcoop map opening with only the co-ops that are members of the NCBA displayed The OBO map can be configured to have only the Community Enterprises visible in the map, but the user can change the filter and show all organisations or just the Support Organisations.

ColmDC commented 9 months ago

Obviously this could be set in the config file and not just specified via a url argument.

ColmDC commented 9 months ago

dependent on https://github.com/DigitalCommons/dotcoop-project/issues/64

lin-d-hop commented 6 months ago

Hi @ColmDC

It sounds like this feature request is the chosen solution to a specific user need. Just for clarity, is the actual use need something like - As someone hosting a mykomap I need to be able to embed a map with a specific location highlighted? and/or As a map user I want to be able to share a specific location using a mykomap

anything else?

Thanks :)

ColmDC commented 6 months ago

The user requirements weren't related to locations. (It is already possible to configure opening the map zoomed and panned to a specific location.) But to use your phrasing they would sound like.

As someone hosting a mykomap I need to be able to embed a map which displays only a subset of the co-ops in the directory when the map opens. For example, I would like to be able to embed the map https://coopsuk.solidarityeconomy.coop/ in an iframe where it only displays the worker co-ops when it first runs up. But if I select Clear Filters, all the the co-ops in the UK will appear.

ColmDC commented 5 months ago

Do you think that needs more clarification @lin-d-hop or can we move it to Dev Ready?

lin-d-hop commented 5 months ago

I'll try chat with @wu-lee to get clear on the spec on this one and will update asap. Right now Nick could certainly do this issue, but it is not clear enough for anyone else to take on.

ColmDC commented 5 months ago

See simplification of the requirements for this feature in description and a change of Acceptance criteria to work on OBO map rather than DotCoop.

wu-lee commented 5 months ago

The user requirements weren't related to locations. (It is already possible to configure opening the map zoomed and panned to a specific location.)

Clarification - location yes, pin, no.

This can be done via config and overridden in the URL. See https://github.com/DigitalCommons/mykomap/blob/master/CONFIG.md

[edit]

Might be better to look at the dev branch docs, as I've now removed the out-of-date explanations of config.json and put in something which describes how it works now.

https://github.com/DigitalCommons/mykomap/blob/dev/CONFIG.md

wu-lee commented 4 months ago

This is now implemented, merged into dev and deployed on https://dev.maps.coop/workers-coop/ for inspection. (Merging immediately since I'm rushing a bit, I am away next week)

The workers-coop data feed had been filtering out non-live co-ops from the data up to now - I've now also removed that filter on the development feed, so they should be appearing within the next 5-10 minutes.

wu-lee commented 4 months ago

The Mykomap documentation currently doesn't have a section describing how supplemental fields are added in any detail (there is an example, but little explanation, see CONFIG.md on the current development branch).

That needs to be added, which should then include documentation for how the filters and their defaults are set.

ColmDC commented 4 months ago

Mykomap documentation currently doesn't have...

Have created a placeholder ticket for this @wu-lee.

ColmDC commented 4 months ago

The workers-coop data feed had been filtering out non-live co-ops from the data up to now - I've now also removed that filter on the development feed, so they should be appearing within the next 5-10 minutes.

That is working nicely now.

ColmDC commented 4 months ago

The search history seems to be getting nobbled now when applying two filters in a row via the Directory. Not observed when using Serach panel.

wu-lee commented 4 months ago

So far as I can see, the directory doesn't really play well with the history. One of the things I found was that the directory would nobble the defaults by simply clearing all the filters when it was opened. When you go back to the filters, the defaults are gone.

This is I think partly a consequence of the directory using a different (older) mechanism for selecting initiatives in the map. I suggested elsewhere we might want to refactor it to use the new mechanism, then it might play better (although we might need to think about what it is expected to do)

Anyway, to make it a bit less weird I altered the behaviour to reset the history when the directory panel is opened/closed, instead of just clearing the filters. This then puts the filters back to the default state when you return.

ColmDC commented 4 months ago

Anyway, to make it a bit less weird I altered the behaviour to reset the history when the directory panel is opened/closed, instead of just clearing the filters. This then puts the filters back to the default state when you return.

So its not a new problem introduced by this function. So shouldn't stop us closing this issue.

After https://github.com/DigitalCommons/mykomap/issues/230 we can take a fresh look at how the functionality feels to use and decide if/when to schedule refactoring the directory it to use the new mechanism.

ColmDC commented 4 months ago

So this function has been demonstrated working as expected on on Workers.coop. Let's test it on the other example in the Acceptance Criteria,

"The OBO map can be configured to have only the Community Enterprises visible in the map, but the user can change the filter and show all organisations or just the Support Organisations."

wu-lee commented 4 months ago

Ok, I've bumped the mykomap version used in the obo-public map to 3.1.6 and added the config. Result deployed here:

https://dev.maps.solidarityeconomy.coop/obo-public/

ColmDC commented 4 months ago

Great.