Skip to content

The trades system#53

Draft
mkutay wants to merge 12 commits intomainfrom
trades-system
Draft

The trades system#53
mkutay wants to merge 12 commits intomainfrom
trades-system

Conversation

@mkutay
Copy link
Owner

@mkutay mkutay commented Jul 24, 2025

  • The main idea for the trades system is complete (with the database tables).
  • Created how thingies are stored for characters (with a doubly linked list to store the edited values).

@coolify-mkutay
Copy link

coolify-mkutay bot commented Jul 24, 2025

The preview deployment for kcldnd.uk failed. 🔴

Open Build Logs

Last updated at: 2025-12-09 14:52:26 CET

@mkutay
Copy link
Owner Author

mkutay commented Jul 24, 2025

Some thinking goes a long way, huh? Instead of having next and previous pointers in the thingy table, we can just have the next so we can quickly get the latest valid version, while also being able to get the previous version by checking the next values according to the current one's id.

@gitguardian
Copy link

gitguardian bot commented Jul 24, 2025

️✅ There are no secrets present in this pull request anymore.

If these secrets were true positive and are still valid, we highly recommend you to revoke them.
While these secrets were previously flagged, we no longer have a reference to the
specific commits where they were detected. Once a secret has been leaked into a git
repository, you should consider it compromised, even if it was deleted immediately.
Find here more information about risks.


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

@mkutay mkutay marked this pull request as draft July 24, 2025 12:30
@mkutay mkutay linked an issue Jul 24, 2025 that may be closed by this pull request
@mkutay mkutay self-assigned this Jul 25, 2025
Copy link
Owner Author

@mkutay mkutay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:(

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.

Magic item economy

1 participant