feat/fix(user ips): fixes and admin panel recent regs widget + v3 catchup#275
Merged
ScuffedNewt merged 96 commits intoScuffedNewt:extension/user-ipsfrom Feb 24, 2026
Merged
Conversation
* Bug(Comments): Permalinks pull all comments into memory Permalinked comments were pulling all comments into memory before finding the comment actually linked to, which will eventually crash the page. * refactor: fix blade formatting --------- Co-authored-by: AW0005 <AW0005@users.noreply.github.com>
…g#1026) * fix(admin): Sales index updated to bootstrap tables * refactor: fix blade formatting * fix(admin): 7 more * refactor: fix blade formatting * fix(admin): Uneven files index columns, thanks to @Moif for pointing it out * fix(admin): Other fixes I should've done --------- Co-authored-by: SpeedyD <SpeedyD@users.noreply.github.com>
…tent (lk-arpg#959) * fix: update image size validation to correctly be 2mb instead of approx. 20mb & add validation to admin reuploads * feat(nit): 2048(kb) -> 2mb & add to update rules (just in case!) * fix(myos): update rules for myos
* fix(admin): Raffle Table messup (Technically I should be cleaning up the indents but I'll let the automation catch that) * refactor: fix blade formatting --------- Co-authored-by: SpeedyD <SpeedyD@users.noreply.github.com>
fix(credits): Update all references to ne-wt to ScuffedNewt
…k-arpg#1066) Co-authored-by: rattusquo <rattusquo@gmail.com>
…arpg#1082) * hotfix: Fix visibility issues for gallery submissions * Update User.php * Requested changes re: scopes * Clearing up a lil bit more redundancy and keeping in line with previous commit * no visibility for you
…ports/mobile (lk-arpg#1099) - User profile rows no longer are so small that they break heading text on >= 992px viewports (as their biggest breakpoint is lg) - Character information pages (image info and stats tab) have also had rows adjusted to prevent breaking headings
* Update GalleryController.php * Update gallery.blade.php
fix: remove extra $ from _profile_content.blade.php (release branch)
- Uses bs-custom-file-input and existing Bootstrap v4 classes to make file inputs that actually show on mobile devices
* Bug(Reports): Fix tabs not appropriately being set to active * Update reports.blade.php * refactor: fix blade formatting --------- Co-authored-by: AW0005 <AW0005@users.noreply.github.com>
feat: update robots.txt
- impacts edit buttons displayed on news and sales posts
…f WYSIWYG comments are enabled
* fix: add captcha block to register_with_driver.blade.php * refactor: fix blade formatting --------- Co-authored-by: perappu <perappu@users.noreply.github.com>
…lk-arpg#1318) - add some minor eager loading
* fix(user profiles): Rank info help does not appear if description empty * I did not think this through * ..Not sure if big improvement? * And we flip back
* fix(character): Minor button renaming * Change 'Edit Notes' to 'Edit Description' * Change 'Edit Features' to 'Edit Traits' link text
- Also selectize on the user dropdown
Owner
I think if there can be a config option to turn it on or off, and some ENV values added to up to you! |
ScuffedNewt
approved these changes
Feb 24, 2026
Owner
ScuffedNewt
left a comment
ScuffedNewt
left a comment
There was a problem hiding this comment.
great PR, thanks for submitting!
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
11 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Had some time last night to put these on a fresh branch finally, but this PR includes the following:
DB::rawquery to display the IP'supdated_atvalue correctly on the indexedit_user_infopower to show the 10 most recent registrations, the corresponding first IP logged on register, as well as if that IP is shared with any existing usersHomeController.phpfor the admin paneledit_user_infopower by defaultSide note: I also have a separate version that will also list if a given IP is a known VPN/proxy or not via using TrustIP Laravel package and its API, but it's not included here as it's an additional package and TrustIp does have a daily maximum quota (500 requests daily, resets at midnight UTC) when on the free plan...on my end I just cache the result so the info doesn't need to be re-fetched all the time from the API but idk how much this would be wanted. 🤔 (And ofc my disclaimer to anyone who is reading this PR: false positives and false negatives are always possible and IPs are largely just checked against known VPN/etc providers, so this is in no way a foolproof method of knowing if a given user is using a VPN or not.)