Skip to content
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

Project ARES: Interface Data Proc #7061

Open
wants to merge 15 commits into
base: master
Choose a base branch
from

Conversation

realforest2001
Copy link
Member

@realforest2001 realforest2001 commented Aug 29, 2024

About the pull request

Adds a centralised proc for getting all the interface data, this will make the UI's much more flexible and massively reduces the amount of duplicate code.

Explain why it's good for the game

Code efficiency, allowing future flexible development.

Testing Photographs and Procedure

Screenshots & Videos

Put screenshots and videos here with an empty line between the screenshots and the <details> tags.

Changelog

🆑
code: Overhauled the backend code for the various ARES related UIs, centralising most interface data to one proc.
/:cl:

@realforest2001 realforest2001 added this to the Project ARES milestone Aug 29, 2024
@github-actions github-actions bot added Mapping did you remember to save in tgm format? UI deletes nanoui/html Code Improvement Make the code longer labels Aug 29, 2024
@realforest2001 realforest2001 marked this pull request as draft September 5, 2024 02:33
cm13-github added a commit that referenced this pull request Sep 5, 2024
@realforest2001 realforest2001 marked this pull request as ready for review September 5, 2024 18:44
cm13-github added a commit that referenced this pull request Sep 6, 2024
cm13-github added a commit that referenced this pull request Sep 6, 2024
cm13-github added a commit that referenced this pull request Sep 7, 2024
cm13-github added a commit that referenced this pull request Sep 7, 2024
@MemesLauncher MemesLauncher mentioned this pull request Sep 7, 2024
3 tasks
cm13-github added a commit that referenced this pull request Sep 7, 2024
cm13-github added a commit that referenced this pull request Sep 7, 2024
@Nanu308 Nanu308 added the Mapping Approved adds 500 new dict keys label Sep 7, 2024
cm13-github added a commit that referenced this pull request Sep 8, 2024
cm13-github added a commit that referenced this pull request Sep 9, 2024
cm13-github added a commit that referenced this pull request Sep 9, 2024
cm13-github added a commit that referenced this pull request Sep 10, 2024
cm13-github added a commit that referenced this pull request Sep 10, 2024
@harryob harryob marked this pull request as draft October 5, 2024 11:08
@realforest2001 realforest2001 marked this pull request as ready for review October 16, 2024 15:24
@cm13-github
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@cm13-github cm13-github added the Merge Conflict PR can't be merged because it touched too much code label Oct 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code Improvement Make the code longer Mapping Approved adds 500 new dict keys Mapping did you remember to save in tgm format? Merge Conflict PR can't be merged because it touched too much code UI deletes nanoui/html
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants