TheNinjaRPG icon indicating copy to clipboard operation
TheNinjaRPG copied to clipboard

Fix issue #658: Updates to War System

Open MathiasGruber opened this issue 4 months ago • 4 comments

Pull Request

Work in Progress

License

By making this pull request, I confirm that I have the right to waive copyright and related rights to my contribution, and agree that all copyright and related rights in my contributions are waived, and I acknowledge that the Studie-Tech ApS organization has the copyright to use and modify my contribution for perpetuity.

MathiasGruber avatar Aug 21 '25 12:08 MathiasGruber

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Preview Comments Updated (UTC)
tnr Ignored Ignored Preview Aug 28, 2025 6:41am

vercel[bot] avatar Aug 21 '25 12:08 vercel[bot]

[!IMPORTANT]

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbit review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing Touches
🧪 Generate unit tests
  • [ ] Create PR with unit tests
  • [ ] Post copyable unit tests in a comment
  • [ ] Commit unit tests in branch openhands-fix-issue-658

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share
🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>, please review it.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbit in a new review comment at the desired location with your query.
  • PR comments: Tag @coderabbit in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbit gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbit read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.

Support

Need help? Create a ticket on our support page for assistance with any issues or questions.

CodeRabbit Commands (Invoked using PR/Issue comments)

Type @coderabbit help to get the list of available commands.

Other keywords and placeholders

  • Add @coderabbit ignore or @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbit summary or @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbit or @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Status, Documentation and Community

  • Visit our Status Page to check the current availability of CodeRabbit.
  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

coderabbitai[bot] avatar Aug 21 '25 12:08 coderabbitai[bot]

@MathiasGruber can you check why Vercel isn't deploying anymore?

theeneon avatar Aug 21 '25 16:08 theeneon

  • [ ] Allow Factions to form an alliance with Villages.
  • [ ] Allow Factions to Request Aid from Villages
  • [ ] Allow Factions to create alliance with other Factions
  • [ ] If a village is at war, block a new war from being created. Instead prompt the faction or village to request alliance with the village that started the war. Another option here is to create a Village Health instead of using townhall as the main metric, so whenever a new ar is started a new village health is created for each instance of war. (Maybe we can discuss this) the problem we're having here is that factinos and villages are teaming up on one village where it's 3v1 and the defender basically has to deplete all the attackers' townhalls while one kill on the defender replinishes all the attackers townhall.
  • [ ] War Objectives These are random objectives that pop up every couple hours while at war that will do bonus damage and bonus healing to townhalls

theeneon avatar Sep 25 '25 23:09 theeneon