Skip to content

File Review Page Revamp #1083

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.

Sign up for GitHub

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

Jump to bottom
Draft
Ishfaq-code wants to merge 60 commits into
base: new-main-page-UX
Choose a base branch
from
Draft

File Review Page Revamp #1083

Ishfaq-code wants to merge 60 commits into from

Conversation

@Ishfaq-code
Copy link

@Ishfaq-code Ishfaq-code commented Nov 25, 2025

THIS PR IS IN PROGRESS!

Major functionalities of the File Review Page PR has been added which include:

  1. Uploading files and changing all references at the same
  2. Deleting unused single file from the form
  3. Changing references works on all content items (including modules) except for Quiz Questions and Quiz Answer Feedback
  4. File filtering

This PR is yet to add

  1. New File Form UI
  2. Selective reference changes
  3. Bulk Deletion
  4. Quiz question and Quiz Answer feedback reference changes

Major functionalities CAN be tested but the entire system is NOT finished yet!

mbusch3 added 30 commits September 4, 2025 09:25
@mbusch3
First pass... Table sort of working. Filters not.
b4b0fbe
@mbusch3
Mid-stage commit to switch branches since I don't want to stash.
7b8c8ec
@mbusch3
Back to feature parity! Files are now separated from other issues.
091d242
@mbusch3
Fixes analyzeReport issue that broke everything.
2162c43
@mbusch3
Tidying up code and translations
393be1b
@mbusch3
Updates the Report Screen to show the correct number of errors, fixed…
f2c8254 
… print screen discrepancies.
@mbusch3
Adjusted severity tag to plural in Report printout.
da869ee
@mbusch3
The Report.js analyzeReport now also finds all references to files in…
1d9f34f 
… contentItems. Still needs module searching.
@mbusch3
First pass... Table sort of working. Filters not.
f986621
@mbusch3
Mid-stage commit to switch branches since I don't want to stash.
55fa3c1
@mbusch3
Back to feature parity! Files are now separated from other issues.
35c746f
@mbusch3
Fixes analyzeReport issue that broke everything.
ffe57e5
@mbusch3
Tidying up code and translations
5868195
@mbusch3
Updates the Report Screen to show the correct number of errors, fixed…
bd2b4f1 
… print screen discrepancies.
@mbusch3
Adjusted severity tag to plural in Report printout.
4118234
@mbusch3
The Report.js analyzeReport now also finds all references to files in…
106ff34 
… contentItems. Still needs module searching.
@mbusch3
Merge branch 'file-review-page' of https://github.com/mbusch3/UDOIT-R…
2c097c2 
…eact-18 into file-review-page
@mbusch3
First pass... Table sort of working. Filters not.
0c16a72
@mbusch3
Mid-stage commit to switch branches since I don't want to stash.
1687356
@mbusch3
Back to feature parity! Files are now separated from other issues.
2a95217
@mbusch3
Fixes analyzeReport issue that broke everything.
94760c1
@mbusch3
Tidying up code and translations
56c923a
@mbusch3
Updates the Report Screen to show the correct number of errors, fixed…
2390513 
… print screen discrepancies.
@mbusch3
Adjusted severity tag to plural in Report printout.
e1c198e
@mbusch3
The Report.js analyzeReport now also finds all references to files in…
fb5dc8b 
… contentItems. Still needs module searching.
@mbusch3
First pass... Table sort of working. Filters not.
adab42a
@mbusch3
Mid-stage commit to switch branches since I don't want to stash.
6babd26
@mbusch3
Back to feature parity! Files are now separated from other issues.
6e6842a
@mbusch3
Tidying up code and translations
26c6a1e
@mbusch3
Updates the Report Screen to show the correct number of errors, fixed…
ae0fc2e 
… print screen discrepancies.
mbusch3 and others added 27 commits October 10, 2025 16:43
@mbusch3
The Report.js analyzeReport now also finds all references to files in…
bf823a2 
… contentItems. Still needs module searching.
@mbusch3
Merge branch 'file-review-page' of https://github.com/mbusch3/UDOIT-R…
22594f5 
…eact-18 into file-review-page
@mbusch3
Moved const variables into Settings.js, logic into Report.js from Fix…
e4d04bc 
…Issues and ReviewFiles pages.
@mbusch3
Finished file filters, added file type icons, first pass on file refe…
429831f 
…rence preview window.
@mbusch3
Add and read from default user settings, removes some magic variables.
8152cfe
@mbusch3
First tweaks for File Form (checkbox doesn't function)
da42f63
@mbusch3
Remove Async EqualAccess, files can now save to LMS.
a0050d5
@mbusch3
The Broken commit: Need to pull replacement files live instead of sto…
9f68e71 
…ring data that could be out-of-date.
@Ishfaq-code
posting files and storing new file Ids in DB
19a360c
@Ishfaq-code
replacement file is added into reports while scanning
0094b3c
@Ishfaq-code
view replacement file next to current file
5ea02c6
@Ishfaq-code
new files pre reviewed on upload
89ab5c8
@Ishfaq-code
new file links for contentbody item on frontend
e3cb04d
@Ishfaq-code
Fix: Report contains file content
ab3c72b
@Ishfaq-code
working route for updating content
8d110cf
@Ishfaq-code
Upload Content API with Multithreading CURL
f2d8701
@Ishfaq-code
working change-all-references tick
ff7540f
@Ishfaq-code
basic file deletion
2a86b8f
@Ishfaq-code
deletion confirm modal
f24ed7d
@Ishfaq-code
added icons and banner to delete modal
6842c50
@Ishfaq-code
File replacement in module - posting and deleting batch
663a0c6
@Ishfaq-code
modules replacement after upload
bd63395
@Ishfaq-code
differentiation files to issues using session files
cd398e1
@Ishfaq-code
improved deletion logic
178c82b
@Ishfaq-code
better error handling and processing upload+reference changes
c170f77
@Ishfaq-code
filtering modules fix
0cb3753
@Ishfaq-code
Active Session File fix - disables save button and works with 'file s…
6b5679f 
…tates'
@Ishfaq-code Ishfaq-code marked this pull request as draft December 1, 2025 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Ishfaq-code @mbusch3