Skip to content

Lots of Minor Updates #114

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
Merged
cycomachead merged 18 commits into from
Aug 25, 2025
Merged

Lots of Minor Updates #114

cycomachead merged 18 commits into from
Aug 25, 2025

Conversation

@cycomachead
Copy link
Contributor

@cycomachead cycomachead commented Aug 23, 2025

Important Stuff:

  • sync 100 assignments (will do full depagination soon, but it's a bit more work. I hope not too many courses have 100 assignments)
  • use a dedicated staff user to process auto-approvals. this really needs a test case.

Future Setup

  • Data Model -- prep for admin? user
  • Data Model -- prep to store most recent assignment/enrollment sync info on course_to_lms
  • Date Model -- better name for students with additional extension time (and started to improve UI a bit)
  • Lots of little refactorings
  • added a for_user() method to CanvasFacade to prevent directly passing the token everywhere.

@cycomachead
Quick fix: Fetch 100 assignments
ef32dd7
@cycomachead
Create a default false admin flag
2ae269f
@cycomachead
Data Model Updates -- Rename dsp column
385c15e 
* auto_approve_extended_request_days is the new setting name
* prepare for storing options on course enrollments
* add sync into to course_to_lms, but currently not stored
@cycomachead
cleanup/tidying
aad3763
@cycomachead
Fixup rename dsp column, clear up UI
e2bdb4c
@cycomachead
Fix autoapprove to use the first staff user in a course
bc4560b
@cycomachead
sigh, fix .tool-versions version
4d7e875
@cycomachead
Small touches: Better footer, tidy documentation
0ea7226
@cycomachead
Move strong_migrations gem, better error message for logged out
8905b85
@cycomachead
lots of refactoring of the existing test setup
34d4eff
@cycomachead
Some progress with a bit of test refactoring
9c1df4b
@cycomachead
delint, no :focus
ab2d5ef
@cycomachead
I think rspec is fully passing...
7be7c1f
@cycomachead
minor progress fixing up Cuke config + small delints
a3075d6
@cycomachead
progress on cuke test cleanup
e202550
@cycomachead
Cleanup Cuke / database cleaner setup a bit
e32a3cf 
This needs a lot more consistency, but tests are again passing
@cycomachead
Test refactoring complete?
3d5bab3
@cycomachead
small system user refactoring
2834531
@cycomachead cycomachead merged commit 805e8a2 into main Aug 25, 2025
11 checks passed
@cycomachead cycomachead deleted the minor-updates branch August 25, 2025 07:03
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

@cycomachead