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

initial whynot_awards #66

Merged
merged 42 commits into from
Jul 20, 2024
Merged

initial whynot_awards #66

merged 42 commits into from
Jul 20, 2024

Conversation

bell07
Copy link
Collaborator

@bell07 bell07 commented Feb 23, 2022

Initial whynot_awards mod

  • Hide all default awards to secret
  • Add new award for sleeping in bed once

Please pull additional awards directly to whynot_awards branch. I think we need some "base" before we can merge it into main branch

@bell07 bell07 mentioned this pull request Feb 23, 2022
@Lazerbeak12345 Lazerbeak12345 marked this pull request as draft August 22, 2022 23:39
@Lazerbeak12345
Copy link
Collaborator

Marked as draft. We need some design-level questions answered first before we keep coding. Ofc if theres something we are in agreement on, like before, just push it.

bell07 and others added 2 commits October 1, 2022 09:38
Just pushing some progress into this....

Co-authored-by: Olivier Dragon <[email protected]>
@dacmot
Copy link
Collaborator

dacmot commented Oct 5, 2022

I've started working on a awards tree diagram with Dia. Let me know what you think.

I'm trying really hard not to put an award for everything :) Seriously though, I feel like it gives a good progression and pointers to get started. I just thought, I forgot to put armor. Some of those achievements will require modifications to the awards mod...

Copy link
Collaborator

@Lazerbeak12345 Lazerbeak12345 left a comment

Choose a reason for hiding this comment

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

Great graph! I like it, though it's sideways.

@dacmot
Copy link
Collaborator

dacmot commented Oct 7, 2022

I tried to print it landscape so it would fit better on a single page... the PDF is more a convenience anyway since the source is the .dia file.

I could remake it top down... might make more sense anyway as we add more awards than can fit on a single page.

@Lazerbeak12345
Copy link
Collaborator

Lazerbeak12345 commented Oct 14, 2022

I'm pretty sure you could just rotate the PDF for now. That's a feature, I'm pretty sure.

@Lazerbeak12345
Copy link
Collaborator

This looks good so far.

@dacmot
Copy link
Collaborator

dacmot commented Feb 26, 2024

This looks good so far.

I haven't been working in this branch directly (yet). It dates back to over a year ago.

I've been toying with different triggers (and gradually adding awards) in https://github.com/dacmot/minetest-game-whynot/tree/new_awards

@dacmot dacmot marked this pull request as ready for review March 19, 2024 23:13
Copy link
Collaborator

@Lazerbeak12345 Lazerbeak12345 left a comment

Choose a reason for hiding this comment

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

It's actually mostly the images I'm worried about.

mods/libs/whynot_awards/custom_triggers.lua Outdated Show resolved Hide resolved
mods/libs/whynot_awards/custom_triggers.lua Outdated Show resolved Hide resolved
mods/libs/whynot_awards/mod.conf Outdated Show resolved Hide resolved
mods/libs/whynot_awards/textures/whynot_awards_apple.png Outdated Show resolved Hide resolved
@dacmot
Copy link
Collaborator

dacmot commented Mar 30, 2024

I also want to add translations (locale) for the awards before merging. I'll do the french one because that's what I know :)

edit: done.

@dacmot
Copy link
Collaborator

dacmot commented Apr 7, 2024

Need to add images for some of the awards.

edit: done.

@dacmot
Copy link
Collaborator

dacmot commented Apr 7, 2024

Need to figure out what's wrong with "Gather and eat wild foodstuff". Doesn't always register something was eaten.

edit: It seems that eating a standalone item (stack=1) won't register eating the item because when we get to the register_on_item_eat callback, the stack is empty and the item name is nil or empty. If the itemstack has two or more items it works fine.

edit2: This does not happen in plain MTG+awards mod. Also, https://github.com/minetest/minetest/blob/e12db0c1828e84693d2fca2eaf2292006767d1f4/builtin/game/item.lua#L404 shows that callbacks are called before removing an item from the stack.

edit3: The culprit seems to be hbhunger. It overrides minetest.do_item_eat and runs the callbacks after taking an item from the stack instead of before like the original function.

edit4: #191

edit 5: merged.

@Lazerbeak12345
Copy link
Collaborator

Is hbhunger even maintained anymore?

@dacmot
Copy link
Collaborator

dacmot commented Apr 8, 2024 via email

@dacmot
Copy link
Collaborator

dacmot commented Jul 18, 2024

I finally got around to adding the last icons for the distance, depth and altitude awards. I'm a terrible graphics artists... so it is what it is. For now. We can maybe ask for help on the forums once it is released.

Speaking of which, I think it's ready for beta testing. @Lazerbeak12345 if you want to give it a shot and let me know what you think, I'd be very grateful.

It's made for the natural progression in survival so not all the awards show up at first. You can use creative to acquire materials quicker, but you still need to follow the order and get the ones displayed in the list.

@Lazerbeak12345 Lazerbeak12345 self-requested a review July 18, 2024 02:59
Copy link
Collaborator

@Lazerbeak12345 Lazerbeak12345 left a comment

Choose a reason for hiding this comment

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

Haven't playtested it yet, but this looks awesome so far!

@dacmot dacmot merged commit ef56a3d into main Jul 20, 2024
@dacmot dacmot deleted the whynot_awards branch August 5, 2024 17:58
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