Skip to content

fix: allows adding npc owner & more - #22

Merged
ErikKalkoken merged 6 commits into
masterfrom
fix-allows-adding-npc-owner
Jul 24, 2026
Merged

ErikKalkoken merged 6 commits into
masterfrom
fix-allows-adding-npc-owner

Conversation

@ErikKalkoken

Copy link
Copy Markdown
Contributor
  • Fixes Don't allow adding NPC owners #17
  • Replaced relative imports
  • Made sure the new admin notification for 403 errors can be configured via setting
  • Various refactoring

@ErikKalkoken
ErikKalkoken merged commit f187ca9 into master Jul 24, 2026
9 checks passed
@ErikKalkoken
ErikKalkoken deleted the fix-allows-adding-npc-owner branch July 24, 2026 13:25
@codecov

codecov Bot commented Jul 24, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 97.72727% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
moonmining/models/owners.py 66.66% 0 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Don't allow adding NPC owners

1 participant