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

Purge non-essential management commands #2580

Closed
1 task done
scottx611x opened this issue Feb 8, 2018 · 3 comments · Fixed by #3332
Closed
1 task done

Purge non-essential management commands #2580

scottx611x opened this issue Feb 8, 2018 · 3 comments · Fixed by #3332

Comments

@scottx611x
Copy link
Member

scottx611x commented Feb 8, 2018

Currently we have a few management commands that we have found to not be needed anymore:

  • @jmarx Could you add the ones you wrote down during our meeting today below?

There are also Management commands whose functionality we still need, but they would be better suited as a data migration.

  • Investigate our current set of management commands
@scottx611x scottx611x added this to the Next milestone Feb 8, 2018
@scottx611x scottx611x added this to Backlog in Technical Debt via automation Feb 8, 2018
@scottx611x
Copy link
Member Author

Ref #2583

@hackdna hackdna moved this from Backlog to Doing in Technical Debt Feb 27, 2018
@jkmarx
Copy link
Member

jkmarx commented Mar 14, 2018

@scottx611x We discussed this during the dev meeting. Were there other commands which needed to be reviewed? #2583 removed the ones we agreed to.

@scottx611x
Copy link
Member Author

I think there are still a few hanging around that we probably don't need:

core/management/commands/install_solr_synonym_plugin.py
core/management/commands/solr_synonyms.py
core/management/commands/uninstall_solr_synonym_plugin.py

@jkmarx jkmarx removed this from the Next milestone Apr 9, 2018
@jkmarx jkmarx added this to the Release 1.6.9 milestone Apr 18, 2019
@jkmarx jkmarx self-assigned this Apr 18, 2019
Technical Debt automation moved this from Doing to Done Apr 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging a pull request may close this issue.

2 participants