Skip to content

feat(admin): add remaining server settings and admin tools (#167)#1503

Open
tiweb442 wants to merge 7 commits into
lanedirt:mainfrom
tiweb442:feature/167-admin-system-options
Open

feat(admin): add remaining server settings and admin tools (#167)#1503
tiweb442 wants to merge 7 commits into
lanedirt:mainfrom
tiweb442:feature/167-admin-system-options

Conversation

@tiweb442

@tiweb442 tiweb442 commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Adds remaining admin server settings from #167: espionage probe capacity ON/OFF, deuterium consumption multiplier (0.5–1.0), systems per galaxy, galaxy option 4, and fixes initial Dark Matter / universe name persistence.
  • Wires universe bounds through galaxy navigation, fleet UI/validation, phalanx, planet relocation, and distance calculations so settings are honored end-to-end (not just stored).
  • Adds admin Activity logs (fleets/buildings/shipyard/research) and Cron tasks pages (list schedule + run allowed scheduler commands).

Test plan

  • As admin, set systems per galaxy to 100, save, hard-refresh galaxy page — system arrows wrap at 100, not 499
  • Set deuterium consumption to 0.5 — fleet fuel preview and actual consumption both drop by ~half
  • Enable espionage probe capacity — probes show cargo 5 in fleet/tech; disable returns to 0
  • Change Initial Dark Matter — new registrations receive that amount
  • Change universe name — admin save persists; page title / meta / settings overlay update
  • Visit /admin/activity-logs and /admin/cron-tasks as admin; confirm non-admins are redirected
  • Attempt planet relocate / phalanx / missile attack outside configured galaxy/system bounds — rejected
  • php artisan test --filter='AdminServerSettingsOptionsTest|AdminTest|UnitCollectionTest|PlanetMoveTest'

Made with Cursor

root @cursoragent
feat(admin): add remaining server settings and admin tools (lanedirt#167
836df54 
)

Wire configurable probe capacity, deuterium consumption, systems per galaxy,
and fix initial dark matter / universe name persistence. Add activity logs
and cron task admin pages, and honor universe bounds across galaxy, fleet,
phalanx, and planet relocation UIs.

Co-authored-by: Cursor <cursoragent@cursor.com>
root @cursoragent
fix(admin): load schedule definitions on cron tasks page
6d40d93 
Schedule events from routes/console.php are only registered during Artisan
bootstrap, so the web admin UI saw an empty list. Load them explicitly when
serving the cron tasks page.

Co-authored-by: Cursor <cursoragent@cursor.com>
root @cursoragent
style(admin): declutter cron tasks page layout
a5e6d61 
Replace the dense table with spaced task rows and friendlier labels
for schedule cadence and command names.

Co-authored-by: Cursor <cursoragent@cursor.com>
root @cursoragent
fix(ui): show configured universe name on login page
9793450 
Replace the hardcoded translated "1. Universe" option and page title
with SettingsService::universeName().

Co-authored-by: Cursor <cursoragent@cursor.com>
@Geda173

Geda173 commented Jul 24, 2026

Copy link
Copy Markdown
Collaborator

Could you please resolve the merge conflicts in this (and the other) PRs so I can start reviewing them?

@tiweb442

tiweb442 commented Jul 24, 2026 via email

Copy link
Copy Markdown
Contributor Author

root @cursoragent
Merge origin/main into feature/167-admin-system-options
98cf857 
Resolve CoordinateDistanceCalculator conflict by keeping getSystemDistance
from main while using configurable numberOfSystems() for wrap-around.

Co-authored-by: Cursor <cursoragent@cursor.com>
@tiweb442

tiweb442 commented Jul 24, 2026

Copy link
Copy Markdown
Contributor Author

Merge conflicts are resolved on this PR and the other conflicting ones (#1509, #1494) — ready for review.

root @cursoragent
fix: import UniverseConstants to satisfy Rector
e69276d 
Co-authored-by: Cursor <cursoragent@cursor.com>
root @cursoragent
fix: satisfy PHPStan null checks in admin settings tests
64d97a0 
Co-authored-by: Cursor <cursoragent@cursor.com>
@github-actions

github-actions Bot commented Jul 24, 2026

Copy link
Copy Markdown

Preview deployment

Preview environment for this PR is ready for functional testing. Use one of the test accounts below to log in.

Status URL Commit Last deployed
✅ Live https://pr-1503.preview.ogamex.dev 64d97a0 2026-07-24 22:22:09 UTC
Test accounts (click to expand)
Email (login) Password Role Class Planets Description
test1@ogamex.dev test admin Collector 2 Admin, high resources, mining focus
test2@ogamex.dev test player General 1 Military focus, ships & fleet
test3@ogamex.dev test player Discoverer 1 Exploration, probes & cargo ships
test4@ogamex.dev test player None 1 New player, no class selected yet
test5@ogamex.dev test moderator Collector 1 Moderator, balanced setup
test6@ogamex.dev test player General 1 Vacation mode enabled
test7@ogamex.dev test player General 3 Heavy defense across planets
test8@ogamex.dev test player Discoverer 1 Fleet focus, inactive (i) 8 days
test9@ogamex.dev test player Collector 1 Beginner, long inactive (I) 40 days
test10@ogamex.dev test player General 5 Endgame, 90 unread messages

This preview will be automatically destroyed when the PR is closed.

@tiweb442

tiweb442 commented Jul 24, 2026

Copy link
Copy Markdown
Contributor Author

i maxed out my credit on cursor i come back in 8 days sorry :S

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lanedirt lanedirt Awaiting requested review from lanedirt lanedirt is a code owner
@jackbayliss jackbayliss Awaiting requested review from jackbayliss jackbayliss is a code owner
@Geda173 Geda173 Awaiting requested review from Geda173 Geda173 is a code owner

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

@tiweb442 @Geda173