Instructor extending individual deadlines: table doesn't sort by team

[damithc]

v8.29, production

Context: an instructor is in the page that is for extending deadlines.

Problem: when trying to sort by the Team column, the row order stays the same i.e., nothing happens when the up/down icons are clicked. But no such issue with the other columns of the table.

[MaharshiPedu]

Hi @damithc, I could not reproduce this issue. The table does get sorted by Team column.

[damithc]

@MaharshiPedu the case I looked at had a few ongoing extensions already. Perhaps that is the trigger for this bug. Can you try that too?

[MaharshiPedu]

@damithc I tried what you suggested. In spite of giving multiple extensions, the order of sorting of Team column did change by using the arrows beside the column heading.

However, I found that the sorting order is wrong. In ascending order, the order is incorrect. One Team 3 row appears between other Team 2 rows. Have a look at the gif below to get a clear understanding:

[damithc]

@MaharshiPedu Also note that the bug I reported is in the production version which may be a number of commits behind the master branch. So, the behavior you see may be different from the one I reported.

@cedricongjh is this behavior potentially affected by recent PRs that we merged? I remember there was a major PR that was related to table sorting.

[cedricongjh]

yes there was this PR: #12556 that was merged in since the last release, hence the difference in behaviour, it could be worth looking into why the team sort (and the other sorts as well) has an issue when sorted by ascending as @MaharshiPedu pointed out

[rameshmoorjani]

Hello @damithc - I am beginner in open source contribution. Can I participate and work on this issue?

[cedricongjh]

hey @rameshmoorjani, feel free to look into this and open a PR

[Ruimeng-Ye]

Hi @damithc , I am a beginner and also interested in this issue, would you mind me working on this issue and trying to fix it?

[wanqiqiu]

hi, @cedricongjh can I work on this issue to have a try? Thank you!
(uid: u7503408 for reference.)

[cedricongjh]

hi @Ruimeng-Ye and @wanqiqiu, since @rameshmoorjani has expressed interest in this issue, let's give him time to work on it, you're welcome to look into other issues to contribute to!

@rameshmoorjani do let us know if you're still working on it, thank you!

[rameshmoorjani]

@cedricongjh - yes I am working on it, having some issue with setup in local due to ng-in-viewport and Agular version mismatch. I will update asap.