Skip to content

Add a tag to specify the reason why a project was blacklisted. #4962

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

Closed

Conversation

oliviersm199
Copy link

  • Added a column in BlackListProjects which allows the user to specify
    the reason that a column was blacklisted from an enumerated type.
  • Created a migration script which defaults already existing blacklisted
    projects to Not Listed
  • Edited the admin panel to allow the user to select a reason when
    creating a new blacklisted project.
  • Edited the admin panel so a user can click on a Blacklisted Project
    and edit the reason/comments.

- Added a column in BlackListProjects which allows the user to specify
the reason that a column was blacklisted from an enumerated type.
- Created a migration script which defaults already existing blacklisted
projects to Not Listed
- Edited the admin panel to allow the user to select a reason when
creating a new blacklisted project.
- Edited the admin panel so a user can click on a Blacklisted Project
and edit the reason/comments.
@oliviersm199
Copy link
Author

#4703 relates to this PR. Going to work on getting unit tests to pass tomorrow.

@di
Copy link
Member

di commented Jun 25, 2019

Hi @oliviersm199, thanks for this PR. This looks mostly good, can you address the test failures and resolve the merge conflicts?

@di di removed request for dstufft and ewdurbin June 25, 2019 22:13
@di
Copy link
Member

di commented Sep 9, 2019

Closing due to inactivity -- @oliviersm199, if you want to finish this, feel free to make a new PR.

@di di closed this Sep 9, 2019
@di di removed their request for review September 9, 2019 19:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants