Skip to content
Snippets Groups Projects

Resolve "Indicate job offer sets 'Head TA' role instead of 'TA'"

Closes #192 (closed) Based on !357 (closed) What it does:

  • Teacher can create HeadTA-only/TA-only/Either roles students can apply to.
  • If teacher changes type of job offer, nothing happens, only when new people accept the job they will be given that new role.
  • Promote/demote button only showed for "Either" type of job offers.
  • Set defaults to "Either" type of job offers (since this is how it has been done before)
  • Testing

Screenshot_2025-02-27_at_10.58.51

Screenshot_2025-02-27_at_10.58.57

Screenshot_2025-02-27_at_10.59.25

Screenshot_2025-02-27_at_10.59.51

Screenshot_2025-02-27_at_11.00.10

Screenshot_2025-02-27_at_11.07.37

Edited by Danae Savvidi

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
  • Ruben Backx requested changes

    requested changes

  • Danae Savvidi added 23 commits

    added 23 commits

    • 936a0c7f...cfbfe74d - 21 commits from branch development
    • d262142a - Merge branch 'development' into 192-indicate-job-offer-sets-head-ta-role-instead-of-ta-2
    • cb71f121 - revert changes

    Compare with previous version

  • Danae Savvidi changed the description

    changed the description

  • Danae Savvidi added 1 commit

    added 1 commit

    Compare with previous version

  • Danae Savvidi added 1 commit

    added 1 commit

    Compare with previous version

  • Ruben Backx resolved all threads

    resolved all threads

  • Danae Savvidi added 4 commits

    added 4 commits

    • 70ee750a...af121958 - 3 commits from branch development
    • e7d9c6e2 - Merge branch 'development' into 192-indicate-job-offer-sets-head-ta-role-instead-of-ta-2

    Compare with previous version

  • Danae Savvidi requested review from @rwbackx

    requested review from @rwbackx

  • Ruben Backx approved this merge request

    approved this merge request

  • Ruben Backx mentioned in commit 4a300df5

    mentioned in commit 4a300df5

  • merged

  • Please register or sign in to reply
    Loading