Skip to content
Snippets Groups Projects

Resolve "Copy lab page"

Merged
Thijs Nullerequested to merge
287-copy-lab-page into development
All threads resolved!

What does this mr do?

Add a copy lab page, which copies over all the current lab's attributes and already pastes them into the create lab form.

Screenshots

before

Screenshot_2020-04-26_at_12.04.08

after

Screenshot_2020-04-26_at_12.02.00

Does this MR meet the acceptance criteria?

  • I have added a changelog entry to reflect the significant changes I made.
  • Tests were created to test the feature.
  • I have updated the documentation accordingly.
  • I adhere to the style guide.

Closes #287 (closed)

Edited by Thijs Nulle

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Chris Lemaire
  • Thijs Nulle added 3 commits

    added 3 commits

    • 8b445ae2 - Add separate controller method for copying lab
    • c2ed25c4 - Merge branch '287-copy-lab-page' of...
    • 45db0a9d - Add tests for LabController.copyLab

    Compare with previous version

  • Thijs Nulle added 207 commits

    added 207 commits

    Compare with previous version

  • Chris Lemaire resolved all threads

    resolved all threads

  • Chris Lemaire approved this merge request

    approved this merge request

  • Thijs Nulle added 27 commits

    added 27 commits

    Compare with previous version

  • Chris Lemaire approved this merge request

    approved this merge request

  • Otto Visser
  • Chris Lemaire resolved all threads

    resolved all threads

  • Otto Visser approved this merge request

    approved this merge request

  • Otto Visser started a merge train

    started a merge train

  • Otto Visser removed this merge request from the merge train because failed to merge. Something went wrong during merge pre-receive hook. Pushes to protected branches that contain changes to files that

    removed this merge request from the merge train because failed to merge. Something went wrong during merge pre-receive hook. Pushes to protected branches that contain changes to files that

  • Otto Visser added this merge request to the merge train at position 2

    added this merge request to the merge train at position 2

  • Otto Visser mentioned in commit 1d8aef2d

    mentioned in commit 1d8aef2d

  • merged

  • unassigned @tnulle

  • Please register or sign in to reply
    Loading