BCStudentSoftwareDevTeam / BCStudentSoftwareDevTeam/lsf

Remove unused function calls from pending forms page

Aperta
#557 1 commento 0 reazioni 1 assegnatario Vedi su GitHub

@johnlolonga19 ci sta già lavorando.

Dal 22/6/2026.

refactor
Lingua principale
Python
Stelle
1
Fork
1
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

We are making javascript function calls on the pending forms page to nonexistent functions. They are mostly not throwing errors, but there isn't a reason for them to exist. Review the Javascript and get rid of old or nonexistent functions.

Example in `app/static/js/allPendingForms.js`:
```
151 // Try and catch is used here to prevent General Search page from reloading the entire the page.
152 try {
153 runformSearchQuery();
154 $('#approvalModal').modal('hide');
155 }
156 catch(e){
157 location.reload(true);
158 }
```
`runformSearchQuery` does not exist in the page.

Review to see if there are other things like this and decide if they need to be replaced with working functionality or if they can just be removed. Also look at surrounding code and clean up as necessary. In this example, we probably don't need a try/catch anymore, and we can just hide the approval modal.

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.