Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Disable and remove scratchpads_col module #6164

Closed
4 of 5 tasks
PaulKiddle opened this issue Mar 5, 2020 · 4 comments · Fixed by #6207
Closed
4 of 5 tasks

Disable and remove scratchpads_col module #6164

PaulKiddle opened this issue Mar 5, 2020 · 4 comments · Fixed by #6207
Labels
type: bug this describes something that isn't working

Comments

@PaulKiddle
Copy link
Contributor

PaulKiddle commented Mar 5, 2020

This is the module that powers the backend col data provider http://classifications.scratchpads.eu. For some reason it is enabled by default on all Scratchpads (dependency of scratchpads_tweaks), however there is no reason to include it. Trying to use the endpoint it provides throws an error, as no sites have the scratchpads_col_terms_ tables in their databases.

Todo:

  • Create new branch & platform for classification.scratchpads.eu
  • Migrate site to new platform
  • Remove scratchpads_col from tweaks dependencies
  • Add migration to uninstall scratchpads_col
  • Remove scratchpads_col files
@PaulKiddle
Copy link
Contributor Author

NB: Create a separate repo/branch/release for classifications.scratchpads.eu, as the only site that uses this module.

@Archilegt
Copy link

@PaulKiddle , this is an essential module in Scratchpads and it shouldn't be removed but fixed and updated. This module gives both new and existing Scratchpads the ability to import a Catalogue of Life classification and to use it as a taxonomic backbone for information aggregation. It is specially important for Scratchpads dealing with large numbers of names. Please, also contact the CoL+ team and proceed to add more annual versions of the CoL. The 2015 versions are way behind from current updates.

@PaulKiddle
Copy link
Contributor Author

PaulKiddle commented Mar 11, 2020

@Archilegt I believe you're talking about the client module scratchpads_classification_service_client, which allows scratchpads to import catalog of life data?
As opposed to scratchpads_col, which is the data provider module and I think is used only by the classifications.scratchpads.eu site, as we only need one data provider.

The former is definitely valuable and I agree should be updated. The latter I think can be removed safely from all standard scratchpads.

@Archilegt
Copy link

@PaulKiddle oki doki!

@alycejenni alycejenni added the type: bug this describes something that isn't working label Mar 16, 2020
@PaulKiddle PaulKiddle reopened this May 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug this describes something that isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants