Skip to content

Misc Bugfixes#735

Merged
itinerare merged 24 commits into
lk-arpg:developfrom
ScuffedNewt:bugfixes
Dec 10, 2023
Merged

Misc Bugfixes#735
itinerare merged 24 commits into
lk-arpg:developfrom
ScuffedNewt:bugfixes

Conversation

@ScuffedNewt
Copy link
Copy Markdown
Contributor

contains a series of bug fixes and some QoL changes

issues mentioned in commits

Ne-wt and others added 3 commits December 6, 2023 01:42
…bugfixes

# Conflicts:
#	app/Services/SubmissionManager.php
#	resources/views/home/_submission_content.blade.php
#	resources/views/home/create_submission.blade.php
Comment thread app/Models/Comment/Comment.php
Copy link
Copy Markdown
Member

@itinerare itinerare left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like there are a few small tweaks need making, but broadly, looks good! Thanks again for taking the time to do this, too.

Comment thread app/Models/Report/Report.php Outdated
Comment thread app/Models/User/User.php Outdated
Comment thread resources/views/auth/register.blade.php
Comment thread resources/views/sales/_sales.blade.php Outdated
Copy link
Copy Markdown
Contributor

@preimpression preimpression left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Commented on a couple little things that probably wouldn't do anything but make me happy lol

Comment thread app/Services/UserService.php
Comment thread resources/views/admin/sales/create_edit_sales.blade.php
Comment thread resources/views/admin/submissions/submission.blade.php
Comment thread resources/views/home/_submission_content.blade.php
@itinerare itinerare added reviewed Pull requests that have received community review and are pending merge and removed needs review Pull requests that are pending community review labels Dec 10, 2023
@itinerare itinerare merged commit 343035e into lk-arpg:develop Dec 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request reviewed Pull requests that have received community review and are pending merge

Projects

None yet

3 participants