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

Bug: the Closed/Open state should appear for Assigned exercises #534

Closed
erikmd opened this issue Apr 7, 2023 · 1 comment · Fixed by #559, #561 or ocaml/opam-repository#24739
Closed
Assignees

Comments

@erikmd
Copy link
Member

erikmd commented Apr 7, 2023

Related user(s):

@erikmd, @AltGr

Related issue(s) or PR(s):

No response

Related project scope(s):

teacher dashboard

Bug description:

If an exercise has at least one assigned student, the teacher dashboard displays the Assigned state only, while it should display Open Assigned or Closed Assigned, to show what the exercise status is for unassigned students.

To reproduce:

No response

Expected behavior:

No response

learn-ocaml --version

No response

git describe --long --always --abbrev=40 --tags

No response

What OS do you use?

GNU/Linux

What OS version/distribution do you use?

Debian

What browser(s) do you use with learn-ocaml?

Firefox, Chromium

What browser(s) version did you used to reproduce the issue?

No response

Screenshots (if need be):

No response

Additional context:

No response

@erikmd
Copy link
Member Author

erikmd commented Jul 28, 2023

@AltGr I keep this issue open because it seems not yet fixed in master
--- tested with the GHA artifacts https://github.com/ocaml-sf/learn-ocaml/actions/runs/5686137220#artifacts from a1dae04

@erikmd erikmd added this to the learn-ocaml 1.0.0 milestone Jul 28, 2023
erikmd added a commit to pfitaxel/learn-ocaml that referenced this issue Aug 25, 2023
* Refactor and fix "Open"/"Closed"/… display code,
  Put function `is_open_or_assigned_globally` in `Learnocaml_data`, returning
  ( GloballyOpen | GloballyOpenOrAssigned
  | GloballyClosedOrAssigned | GloballyClosed )

* Make invariants explicit in OCaml comments

Close ocaml-sf#534
Close ocaml-sf#558
erikmd added a commit to pfitaxel/learn-ocaml that referenced this issue Aug 25, 2023
* Refactor and fix "Open"/"Closed"/… display code,
  Put function `is_open_or_assigned_globally` in `Learnocaml_data`, returning
  ( GloballyOpen | GloballyOpenOrAssigned
  | GloballyClosedOrAssigned | GloballyClosed )

* Make invariants explicit in OCaml comments

Close ocaml-sf#534
Close ocaml-sf#558
erikmd added a commit to pfitaxel/learn-ocaml that referenced this issue Aug 25, 2023
* Refactor and fix "Open"/"Closed"/… display code,
  Put function `is_open_or_assigned_globally` in `Learnocaml_data`, returning
  ( GloballyOpen | GloballyOpenOrAssigned
  | GloballyClosedOrAssigned | GloballyClosed )

* Make invariants explicit in OCaml comments

Close ocaml-sf#534
Close ocaml-sf#558
erikmd added a commit to pfitaxel/learn-ocaml that referenced this issue Aug 28, 2023
* Refactor and fix "Open"/"Closed"/… display code

* Add function `is_open_or_assigned_globally` in `Learnocaml_data` that returns
  ( GloballyOpen
  | GloballyOpenOrAssigned
  | GloballyClosedOrAssigned
  | GloballyClosed )

* Make invariants explicit in OCaml comments

* Add `check…` and `fix…` functions in `Learnocaml_data.Exercise.Status`
  to assert if the (assignments map, default) comply with the invariants
  (a false result means that there is at least one Open and one Closed)

Close ocaml-sf#534
Close ocaml-sf#558
erikmd added a commit to pfitaxel/learn-ocaml that referenced this issue Aug 28, 2023
* Refactor and fix "Open"/"Closed"/… display code

* Add function `is_open_or_assigned_globally` in `Learnocaml_data` that returns
  ( GloballyOpen
  | GloballyOpenOrAssigned
  | GloballyClosedOrAssigned
  | GloballyClosed )

* Make invariants explicit in OCaml comments

* Add `check…` and `fix…` functions in `Learnocaml_data.Exercise.Status`
  to assert if the (assignments map, default) comply with the invariants
  (a false result means that there is at least one Open and one Closed)

Close ocaml-sf#534
Close ocaml-sf#558
erikmd added a commit to pfitaxel/learn-ocaml that referenced this issue Sep 10, 2023
* Refactor and fix "Open"/"Closed"/… display code

* Add function `is_open_or_assigned_globally` in `Learnocaml_data` that returns
  ( GloballyOpen
  | GloballyOpenOrAssigned
  | GloballyClosedOrAssigned
  | GloballyClosed )

* Make invariants explicit in OCaml comments

* Add `check…` and `fix…` functions in `Learnocaml_data.Exercise.Status`
  to assert if the (assignments map, default) comply with the invariants
  (a false result means that there is at least one Open and one Closed)

Close ocaml-sf#534
Related: ocaml-sf#558
erikmd added a commit that referenced this issue Sep 15, 2023
fix(teacher_tab): partly fix Open/Close handling w.r.t. Assignments

Close #534
Related: #558
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment