- Add eslint and stylint to pre-commit
- Update .github workflow
- upgrade ruff
- Make mirror-eslint work (ps: it doesn't seem to find the "globals"
package if not installed with npm install explicitly
- Add timeout to each task

Fixes #3957

Fixes the query parameter being show twice while searching twice on the
page.

The issues was that both `<form>` and `hx-include` was adding data when
the form was submitted with "Enter" key.

Fixes #3817 

Fixes #3048 
Fixes #4029

- [x] Introduce machine generated avatars based on the user's name
(in-future allow for gravatar or similar service)
- [x] Color Code the user avatar based on their roles  
- [x] Render the staff roles as label (#3916)
- [x] Reduce the size of the rendered comment box.
- [x]  Merge activities with communication in a single timeline.  
- [x] Use htmx for comment edit function, the js version seems to break
some functionality.
- [x] Use different icons for different activities where possible. 
- [x] Render paginated comment and activities  
- [x] Cleanup/remove JSON APIs related to comments
- [x] Use the same logic for rendering of comments on the projects
module
- [x] ~Async comment add (won't be done in this PR)~
- [x] Deep-link for individual comments
- [x] fix testcases 

Fixes #4158

Fixes #3971

Co-authored-by: Ghulam Murtaza <gmurtaza@opentechstrategies.com>
Co-authored-by: Jesse Bickel <jbickel@opentechstrategies.com>

Fixes #4074 

In some of the translation features I was using optional chaining &
eslint would toss errors saying it was invalid despite it working. This
led me to exploring the config to find the [yaml config we used was
depreciated](https://eslint.org/docs/latest/use/configure/configuration-files-deprecated).
This PR updates to a newer version of eslint & migrates the config to
the [flat file
version](https://eslint.org/docs/latest/use/configure/configuration-files#configuration-file)
(and now optional chaining is recognized ).

Some code changes that came with the updated linting:
- [As of
ES6](https://262.ecma-international.org/6.0/#sec-strict-mode-code)
modules are always in strict mode, thus `"use strict"` isn't needed
- Removed unused var
- Removed unneeded `eslint-disable` comment

There are also a ton of JSDoc warnings now - I was going to remove them
but I think JSDoc is actually a weak point in our code documentation so
maybe the in-terminal annoyances will be a good reminder to do a quick
write up

Right now, the custom templates put in the recommended "template_custom"
doesn't checked by tailwindcss. So, the custom templates are not styled
by tailwindcss.

Fixes #3546

Improves the naming of cloned forms by changing the format
to `<original form name> (Copied on <%Y-%m-%d %H:%M:%S.%f>)`, ie.
cloning `Test Form` would result in a duplicated form with the name
`Test Form (Copied on 2024-08-27 17:34:32.004)`. If a form name with a
timestamp already exists on a form it will be subbed via RegEx matching.

Since the APIs are all depreciated, and APIs are not used by anyone,
this PR starts the depreciation phase and tries to remove the dead load.

Fixes #4144

Fixes #4082 

Fixes #4141

Adds a management command to delete all drafts older than two years.
Intended to be used with a cron job to consistently clean up drafts.

Fixes #4049 

Fixes #3537 

Adds icons & edit links to the Wagtail forms views (Application Forms,
Determination Forms, Review Forms, etc.) "Used by" column, making it
easier to differentiate what's used where at a glance . Sometimes rounds
& funds take the same name so it's not possible to know which is which.

Also added an enum class to utils to allow for consistency across the
interface with icon usage rather than hardcoding the same values over
and over. I would've used `StrEnum` but it felt strange to have repeat
values for different enums. Open to better solutions, though.

Changes

- [x] Replaces the old view with new table view, that is powered by the
filtering and rendering logic build for the new list view.
- [x] removes round detail view and round export, reuse the submission
all page.
- [x] Add rounds/labs menu to primary nav, makes it easy to list and
view status of different rounds, for staff.
- [x] Update my flags view on the dashboard page.
- [x] Implement `SUBMISSION_LIST_EXCLUDE_FIELDS` on both list and table
view (in progress)
- [x] Expose old table view at an accessible url

---------

Co-authored-by: Fredrik Jonsson <frjo@xdeb.org>

- The current make -j parallelization doesn't kill the remaining process
when one of them errors out, the causes confusion if let's css/js is not
building.

- This PR also refractors the help target to use comment based generator
which is more maintainable.

- Update the node scripts to not require `collectstatic` during
development, the runserver -> static server can find all the all files
from packages and serve them on demand.
- `make serve` now also installs python & node packages and builds the
static_src if they are outdated.

---------

Co-authored-by: Fredrik Jonsson <frjo@xdeb.org>

Fixes #4054 

Fixes #4091 

Fixes #4101 

Fixes #4048 

Fixes #4094. 

~~For some reason, when combining other triggers with
`revealed` the revealed trigger itself breaks while others remain fine.
Swapping for the `intersect once` trigger resolved this but I'm not
exactly sure why. Also not sure if moving forward for consistency we
should use `intersect` instead of `revealed`. @theskumar any thoughts?~~

Updated htmx from v1.9.12 -> v2.0.2 to fix the usage of multiple tags
breaking `revealed`. Tested locally and there doesn't seem to be any
major breaking changes.

When there is no related submissions the script generates an error. This
PR fixes that.

Also removes one old unused js file.

Fixes #4081 

The applicant dashboard (and various other views) used tailwind's
`bg-light-grey` which seems inconsistent with the overall app and caused
the menu to have a different standout color.

I believe the grey background does add a nice contrast to things so I
made that the system default. Open to making it all white, though.