-
Notifications
You must be signed in to change notification settings - Fork 737
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
Dogtags - Rename inventory items #10102
Conversation
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.
IMO this should be a framework
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.
I agree, but frankly I don't want to invest the time and effort into making a framework. I saw the CBA one, but that seems to be stale.
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.
It was mostly done at the time iirc, just needs someone to wrap it up and make any changes that need to be accounted for
acre does something similar and struggles with flickering - IDI-Systems/acre2#1256 still fully usable with the ficker, but if we could find a solution it would be a benefit to both |
Co-authored-by: PabstMirror <pabstmirror@gmail.com>
Closed in favor of #10130. |
When merged this pull request will:
IMPORTANT
Component - Add|Fix|Improve|Change|Make|Remove {changes}
.