-
Notifications
You must be signed in to change notification settings - Fork 29
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
refactor: Diplomacy and advisor icon refactor to new image system #363
base: main
Are you sure you want to change the base?
refactor: Diplomacy and advisor icon refactor to new image system #363
Conversation
updated advisors to new image handler added ability to define custom advisor images through chapter json
Reviewer's Guide by SourceryThis pull request refactors the diplomacy and advisor icons to use the new image system. It also adds the ability to define custom image IDs for advisors in the chapter JSON files. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @VanWeapon - I've reviewed your changes and they look great!
Here's what I looked at during the review
- 🟢 General issues: all looks good
- 🟢 Security: all looks good
- 🟢 Testing: all looks good
- 🟢 Complexity: all looks good
- 🟢 Documentation: all looks good
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
also just because this is a potential annoyance change (cm code being as it is) will hold back for 10.1 just to help push 0.10.0 over the line |
@sourcery-ai review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @VanWeapon - I've reviewed your changes and they look great!
Here's what I looked at during the review
- 🟢 General issues: all looks good
- 🟢 Security: all looks good
- 🟢 Testing: all looks good
- 🟢 Complexity: all looks good
- 🟢 Documentation: all looks good
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
Description of changes
Reasons for changes
Related links
Summary by Sourcery
Refactor diplomacy and advisor icons to use the new image system. Allow custom image IDs for advisors in chapter JSON files.
New Features:
Enhancements: