Skip to content

Merge Reports into modified main#36

Merged
itinerare merged 32 commits into
lk-arpg:modified-mainfrom
ScuffedNewt:reports
Nov 6, 2020
Merged

Merge Reports into modified main#36
itinerare merged 32 commits into
lk-arpg:modified-mainfrom
ScuffedNewt:reports

Conversation

@ScuffedNewt
Copy link
Copy Markdown
Contributor

merge reports addon into modified main - will allow reportable compatibility with comments, galleries etc in future

ScuffedNewt and others added 27 commits October 18, 2020 11:27
# Conflicts:
#	.gitignore
#	resources/views/admin/index.blade.php
…into reports

# Conflicts:
#	.gitignore
#	app/Http/Controllers/Comments/CommentController.php
#	routes/lorekeeper/browse.php
…eports

# Conflicts:
#	app/Models/Notification.php
#	config/lorekeeper/notifications.php
…into reports

# Conflicts:
#	app/Models/Notification.php
#	config/lorekeeper/notifications.php
- Add report sidebar links to admin and home
- Adjust admin index page for consistency
- Add bug report index link to footer
- Add an additional type of bug, "exploit", and make only bugs of this type be hidden in the general bug index
- Fix bug re user commenting on an unassigned report
… into reports

# Conflicts:
#	app/Http/Controllers/Comments/CommentController.php
#	app/Models/Notification.php
#	config/lorekeeper/notifications.php
#	resources/views/admin/reports/report.blade.php
#	resources/views/home/_report.blade.php
#	resources/views/home/_report_content.blade.php
#	resources/views/home/bug_report_index.blade.php
@itinerare itinerare added the needs review Pull requests that are pending community review label Nov 5, 2020
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.

I've already gone through and tested + done a fixes pass, plus given everything another once-over here

Copy link
Copy Markdown
Collaborator

@Draginraptor Draginraptor 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 to me

@itinerare itinerare merged commit 9442367 into lk-arpg:modified-main Nov 6, 2020
@ScuffedNewt ScuffedNewt deleted the reports branch May 21, 2021 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs review Pull requests that are pending community review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants