Sort of courses on the home page

[ramzieus]

Proposed Changes

• add migration to add sort_index column to Course
• add buttons up and down to control the order of the courses

@pupilfirst/developers
@pupilfirst/designers

Merge Checklist

• Add specs that demonstrate bug / test a new feature.
• Check if route, query, or mutation authorization looks correct.
• Ensure that UI text is kept in I18n files.
• Update developer and product docs, where applicable.
• Prep screenshot or demo video for changelog entry, and attach it to issue.
• Check if new tables or columns that have been added need to be handled in the following services:
• Check if changes in packaged components have been published to npm.
• Add development seeds for new tables.
• If the updates involve Graph mutations ensure that the files are migrated to the new approach without a mutator.
• If the updates involve adding a new table ensure that rate limiting is added and documented in the docs/developers/rate_limiting.md file.

[kaisersakhi]

@ramzieus Could you please complete the default checklist? Also, add specs for this feature as well.

[kaisersakhi]

@vinutv @irajsuhail Could you please have a look at this whenever you're free.

[kaisersakhi]

• ReScript yet to be reviewed

[ramzieus]

@kaisersakhi spec added

[ramzieus]

@yash-learner done

[irajsuhail]

Please check the updated design for the admin dashboard
Figma doc

@ramzieus @yash-learner @vinutv

[ramzieus]

@irajsuhail good and useful design

[harigopal]

Either disable the downward direction swap for last course in the loaded courses with the first course of unloaded courses, or Handle the UI when swap is allowed.

@yash-learner It looks like this is already managed in the UI? The top course only shows the down arrow, and the bottom course only shows the up arrow. 🤔

Auth token is allowed for move_course mutation, Is this required?

There doesn't seem any harm in it. 🤷🏼‍♂️

We should also check all listing of courses to see if it follows the sort index;

I think we can gradually apply the use of courses.sort_index.

Also check how the sort index will be affected if we have applied filters.

It shouldn't have any special effect. Even when any filter is applied, the resulting list needs to be ordered by something - that something is the sort_index now.

It will be great if we have a public filter, that allows users to see all publicly listed courses.

Not absolutely necessary to get this merged, I think. Skip for now?

[yash-learner]

Comments on latest changes

It looks like this is already managed in the UI? The top course only shows the down arrow, and the bottom course only shows the up arrow. 🤔

Not handled in UI when the page is paginated

We decided not to swap the last course in the loaded courses with the first course of unloaded courses in pagination.

---

It shouldn't have any special effect. Even when any filter is applied, the resulting list needs to be ordered by something - that something is the sort_index now.

Yes

---

Not absolutely necessary to get this merged, I think. Skip for now?

Yes

I think we can gradually apply the use of courses.sort_index.

Yes

[harigopal]

Not handled in UI when the page is paginated

@yash-learner I opted not to show the Move down option for the last entry in the list, regardless of whether it's fully loaded or partially loaded. This small change allows the existing UI code to work as intended. The Move down option will appear on last loaded entry if more are loaded beneath it.

Stale

[yash-learner]

Thank you, @ramzieus, for the contribution 🙌