Robocraft999 / TraidingNetwork

MIT License
1 stars 2 forks source link

PR No 2 #4

Closed Darudas closed 3 months ago

Darudas commented 3 months ago

I am a fan of the AmazingTrading mod and have been working on some enhancements to help it shine even more in the future. Here are the key improvements I've made:

Enhancements

New Command

I believe these enhancements and fixes will significantly improve the user experience and functionality of the mod. I look forward to your feedback and hope these changes can be integrated into the main project.

Darudas commented 3 months ago

@Robocraft999

Robocraft999 commented 3 months ago

So first of all I'm sorry that I let you wait and thanks for your work. But there are a few things I would change. The first thing is to maybe move the sucking of items to a seperate Block like a hopping shredder (extending the current). Furthermore there are some weird ways you try to get the resourcepoint values. The recipes are very nice though. The shredding of items without value I'm not sure about. I thought about maybe putting it beheind a config or something but it kind of breaks with the concept of only being to crush predefined items and not really being a trashcan. The command thing i would move to a seperate pr. I also will have to have a look at the displaying before merging. Also I have to get the fluid integration working I'm working on right now to release it alltogether as 0.2, but it's a bit broken right now.

Robocraft999 commented 3 months ago

So i had a look now. The Item displaying looks very nice, the recipes are nice, but I don't understand why to add this command. Also the sucking seems only to work if the inventory is empty and single items sometimes don't get displayed.

Darudas commented 3 months ago

and I would love to talk to you in discord a bit about the mod to get a better understanding of what you want and what you dont want in the mod in the future and I would love to share you some of my workflows and would love to know what you think about it

Robocraft999 commented 3 months ago

merge and remove or edit whatever you want and I will than work furter with your edits and sorry for all the leftovers as I am kinda new to git and still do most stuff the oldscool way with my limited knowladge and thats how some stuff gets removed or left from a side project with the code, I should start using branches xD

Don't worry that's the same for me. I'm still afraid of using branches for everything and so on, but I think eventually I'll get used to it. (Also first time using this review feature of github for this pr). Regarding the discord thing, sure. Just contact me.

Darudas commented 3 months ago

Ive send you a friend requests but because of your privacy settings I can't send you a massage