Skip to content

File review page - Initial Commit #1075

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
Open
mbusch3 wants to merge 40 commits into
base: new-main-page-UX
Choose a base branch
from
Open

File review page - Initial Commit #1075

mbusch3 wants to merge 40 commits into from

Conversation

@mbusch3
Copy link
Contributor

@mbusch3 mbusch3 commented Oct 31, 2025

Still to do:

  • Make the FileItem.php file return a "replacement" object IF it exists in the local database.
  • Allow users to save the links to the replacement file over those of the old file.
  • Allow users to see the old and replacement files side-by-side and manually change the references AFTER the file upload process.

Lots already in place: new page, divorced FixIssuesPage.js from ReviewFilesPage.js, LOTS of cleaning of old code as well.

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 added 10 commits October 10, 2025 16:43
@mbusch3
Adjusted severity tag to plural in Report printout.
f8d58aa
@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.
@mbusch3
Merge branch 'new-main-page-UX' into file-review-page
8f172cd
@dmols dmols self-requested a review December 2, 2025 21:14
@dmols dmols self-assigned this Dec 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dmols dmols Awaiting requested review from dmols

Assignees

@dmols dmols

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mbusch3 @dmols