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

Implement mall (itemshop storage) #121

Draft
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

WoozChucky
Copy link
Collaborator

@WoozChucky WoozChucky commented May 11, 2024

This still very much a work in progress, still need to discuss if we're going to have a common api interface (+ abstract class?) with the other inventory windows or not. Same for some of the Item models and packets fields naming.

Tasks:

  • Open mall
  • Close mall
  • Send mall items to client (real implementaion is lacking, but we can send a hardcoded item as a PoC)
  • Db + schema (out fo scope for now)
  • Move item from mall (out fo scope for now)
  • Tests

@MeikelLP
Copy link
Owner

Wouldn't this be a HTTP API app? Because we need some kind of frontend too?

Wouldn't this be another standalone application?

Also I don't know how important the in game shop is for now 👀 maybe kinda out of scope for now?

@WoozChucky
Copy link
Collaborator Author

Wouldn't this be a HTTP API app? Because we need some kind of frontend too?

Wouldn't this be another standalone application?

Also I don't know how important the in game shop is for now 👀 maybe kinda out of scope for now?

Yes I agree. I tried to cleanup the pr as much as possible.
So basic functionallity included now is to open and close the mall without password validation (harcoded to 000000).

If you agree that this could/should be merged in, we can do it, and then at a later stage we can implement the rest of the funcionalities for the mall.

@WoozChucky WoozChucky self-assigned this May 13, 2024
@WoozChucky WoozChucky force-pushed the feature/open-ingame-mall branch from 22ec2c3 to 0d7358b Compare June 14, 2024 01:18
@eleite93
Copy link

If you want to maintain the dotnet stack, I would really like to give a helping hand in implementing some dotnet metin2 CMS using Blazor/MudBlazor

@MeikelLP
Copy link
Owner

MeikelLP commented Jun 20, 2024

If you want to maintain the dotnet stack, I would really like to give a helping hand in implementing some dotnet metin2 CMS using Blazor/MudBlazor

@eleite93

We are not bound to just .NET

On the other side I personally dislike Blazor. I have used it and would always pefer simple Razor views. I would not want to maintain such a technologie in my stack. However I wholeheartedly welcome you to implement it yourself as third party tool :)

Side note: Ask this in the discord or in a Diskussion (if available?)

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.

3 participants