Skip to content

Implement corpus delete logic

- requested to merge 345089-corpus-row-delete into master

What does this MR do and why?

Adds ability to delete a corpus from the corpus management screen #345089 (closed)

Describe in detail what your merge request does and why.

Screenshots or screen recordings

Screen_Shot_2022-01-10_at_5.03.34_PM Screen_Shot_2022-01-10_at_5.03.21_PM Screen_Shot_2022-01-10_at_5.03.26_PM

How to set up and validate locally

Validation errors will be handled in #342583

  • From the gitlab repo open a rails console and enable the :corpus_management AND :corpus_management_ui feature flag
bundle exec rails c

then

[4] pry(main)> Feature.enable(:corpus_management)
[4] pry(main)> Feature.enable(:corpus_management_ui)

Navigate to:

http://172.16.123.1:3000/<group>/<project>/-/security/configuration/corpus_management

  • Click on New Corpus, upload any zip file

  • Click "Add", should see the corpus added to the table

  • VERIFY table reloads with corpus

  • Click the trashcan item to delete corpus

  • VERIFY you see delete modal

  • VERIFY confirming in modal reloads the table and deletes the corpus

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #345089 (closed)

Edited by Jannik Lehmann

Merge request reports