feat: remove tool sectors

platform-coop-toolkit / coop-map-directory-index

The Map/Directory/Index component of the Platform Cooperative Development Kit.

https://directory.platform.coop

BSD 3-Clause "New" or "Revised" License

1 stars 3 forks source link

feat: remove tool sectors #62

Closed greatislander closed 4 years ago

greatislander commented 4 years ago

[x] I've read the guidelines for Contributing to the Platform Co-op Toolkit
[x] This isn't a duplicate of an existing pull request

Description

This PR removes the sectors field from the Tools model as this data will be determined by a tool's relationship to cooperatives.

Steps to test

Apply the migration.
Edit a tool via the admin interface.

Expected behavior: The sectors field is no longer shown.

Additional information

Not applicable.

Related issues

Depends on #59.

erictheise commented 4 years ago

@greatislander, what happens if a software co-operative creates a new Tool for a specific Sector but no Organization in the Directory is yet using it? Seems the Tool creator should be able to directly specify a Sector for their product.

greatislander commented 4 years ago

Thanks @erictheise, this makes sense— we were grappling with the necessity to filter tools by sector in two different ways (sector the tool was built for, sector or sectors the tool is used in) but @cherylhjli and I just discussed and I think we've come up with a good approach. Closing this PR.