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

Added fishing minigame; Still bugs to fix #137

Merged
merged 14 commits into from
Oct 16, 2024

Conversation

fullspeccoder
Copy link
Contributor

Fishing Minigame

Had to work through some major changes due to major refactor, but managed to figure it out at least for now. I implemented the fishing minigame for the project - but it doesn't work on mobile just yet. I was thinking of maybe implementing that version where the user stays in a spot and has to tap to get a fish.

Implementation

A game that allows you to cast a hook and will catch fish on the way up. In this version, you can move side to side (albeit slowly because that makes it more challenging). The fishes spawn in random places between boundaries with random sprites used from the fish.png image from the sprites folder. I used a strategy of loading assets before the game starts, configuring the assets to be used correctly, then allow the user to play the game.

Things To Improve

Upon successful catches once you leave the minigame, you should be able to keep the fish you caught. I'm not sure what this could look like; whether that means you can keep the specific names of the fish, or keeping the weights. There are so many possibilities!

Will work out details on getting state to work versus the triggerFishingAnimations and triggerNormalAnimations functions.

Hopefully the structure for the minigame won't affect the rest of the code in the project, I tried to modularize it as much as possible away from the rest of the code.

Copy link

🙏 Thanks for your pull request @fullspeccoder, The team will now review and merge this request. In the mean time why not check out some of the other opensource projects available, contributions are greatly appreciated!

Some of the most popular are


PR Statistics

#️⃣ PR Number: Line Additions: 🗑️ Line Deletions:
137 1142 16
📑 Files Changed: Repo Stars: 🔱 Total Forks:
30 27 71

Copy link

🙏 Thanks for your pull request @fullspeccoder, The team will now review and merge this request. In the mean time why not check out some of the other opensource projects available, contributions are greatly appreciated!

Some of the most popular are


PR Statistics

#️⃣ PR Number: Line Additions: 🗑️ Line Deletions:
137 1142 16
📑 Files Changed: Repo Stars: 🔱 Total Forks:
30 27 71

Copy link

🙏 Thanks for your pull request @fullspeccoder, The team will now review and merge this request. In the mean time why not check out some of the other opensource projects available, contributions are greatly appreciated!

Some of the most popular are


PR Statistics

#️⃣ PR Number: Line Additions: 🗑️ Line Deletions:
137 1133 19
📑 Files Changed: Repo Stars: 🔱 Total Forks:
30 27 71

@fullspeccoder
Copy link
Contributor Author

Through the code changes I realized that all I really should have done was pass in a type of fishing scene then create the string scene so that the player could go back to the map they were on.

@r4pt0s
Copy link
Collaborator

r4pt0s commented Oct 11, 2024

@fullspeccoder that is what I meant on Discord. I tried to reduce the boilerplate code for scene addition. Hopefully it will help you to add your part more easily

Copy link

⚠️ MERGE CONFLICT DETECTED!

@fullspeccoder A possible conflict has been detected, you will need to resolve this before your pull request can be merged. The most common reason conflicts occur, is when the contributor does not run git pull origin master before pushing their new changes.

Before we can merge the code, you will need to resolve the conflict, check out these videos by Aldo: How To Get Started With Open Source and How To Resolve Git Merge Conflicts. If you get stuck ask over on Discord.

1 similar comment
Copy link

⚠️ MERGE CONFLICT DETECTED!

@fullspeccoder A possible conflict has been detected, you will need to resolve this before your pull request can be merged. The most common reason conflicts occur, is when the contributor does not run git pull origin master before pushing their new changes.

Before we can merge the code, you will need to resolve the conflict, check out these videos by Aldo: How To Get Started With Open Source and How To Resolve Git Merge Conflicts. If you get stuck ask over on Discord.

Copy link

🙏 Thanks for your pull request @fullspeccoder, The team will now review and merge this request. In the mean time why not check out some of the other opensource projects available, contributions are greatly appreciated!

Some of the most popular are


PR Statistics

#️⃣ PR Number: Line Additions: 🗑️ Line Deletions:
137 1341 276
📑 Files Changed: Repo Stars: 🔱 Total Forks:
30 27 72

1 similar comment
Copy link

🙏 Thanks for your pull request @fullspeccoder, The team will now review and merge this request. In the mean time why not check out some of the other opensource projects available, contributions are greatly appreciated!

Some of the most popular are


PR Statistics

#️⃣ PR Number: Line Additions: 🗑️ Line Deletions:
137 1341 276
📑 Files Changed: Repo Stars: 🔱 Total Forks:
30 27 72

@fullspeccoder
Copy link
Contributor Author

I finally added the minigame! In a similar style to what you've refactored it to @r4pt0s, I have also fixed the scaling but I see that it was also fixed in another PR as well. Fixing the scaling helped with an overall issue I was having with the game. I tested escaping out of the game and it worked okay, the game works as intended, and overall seems to work okay. Let me know if there's any changes!

@r4pt0s
Copy link
Collaborator

r4pt0s commented Oct 12, 2024

Hi @fullspeccoder,

I looked and tested it.
There is a problem with the mobile view of your mini game.
Screenshot_2024-10-12-17-33-45-82_40deb401b9ffe8e1df2f1cc5ba480b12

On desktop everything looks and works as expected.

Also, could you please merge the current main branch into your PR 🙏🏻

Copy link
Collaborator

@r4pt0s r4pt0s left a comment

Choose a reason for hiding this comment

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

^

Copy link

🙏 Thanks for your pull request @fullspeccoder, The team will now review and merge this request. In the mean time why not check out some of the other opensource projects available, contributions are greatly appreciated!

Some of the most popular are


PR Statistics

#️⃣ PR Number: Line Additions: 🗑️ Line Deletions:
137 1570 277
📑 Files Changed: Repo Stars: 🔱 Total Forks:
30 27 78

Copy link

🙏 Thanks for your pull request @fullspeccoder, The team will now review and merge this request. In the mean time why not check out some of the other opensource projects available, contributions are greatly appreciated!

Some of the most popular are


PR Statistics

#️⃣ PR Number: Line Additions: 🗑️ Line Deletions:
137 1537 280
📑 Files Changed: Repo Stars: 🔱 Total Forks:
30 27 78

@fullspeccoder
Copy link
Contributor Author

Hello again! I have finally figured out a way to make the game look good on Mobile, as the screen was scrunched up at the top. Hopefully this is the absolute winner! Let me know if you need any more changes. @r4pt0s

Copy link

🙏 Thanks for your pull request @fullspeccoder, The team will now review and merge this request. In the mean time why not check out some of the other opensource projects available, contributions are greatly appreciated!

Some of the most popular are


PR Statistics

#️⃣ PR Number: Line Additions: 🗑️ Line Deletions:
137 1445 279
📑 Files Changed: Repo Stars: 🔱 Total Forks:
30 27 79

Copy link

🙏 Thanks for your pull request @fullspeccoder, The team will now review and merge this request. In the mean time why not check out some of the other opensource projects available, contributions are greatly appreciated!

Some of the most popular are


PR Statistics

#️⃣ PR Number: Line Additions: 🗑️ Line Deletions:
137 1445 279
📑 Files Changed: Repo Stars: 🔱 Total Forks:
30 27 79

Copy link

🙏 Thanks for your pull request @fullspeccoder, The team will now review and merge this request. In the mean time why not check out some of the other opensource projects available, contributions are greatly appreciated!

Some of the most popular are


PR Statistics

#️⃣ PR Number: Line Additions: 🗑️ Line Deletions:
137 1444 279
📑 Files Changed: Repo Stars: 🔱 Total Forks:
30 27 79

Copy link

🙏 Thanks for your pull request @fullspeccoder, The team will now review and merge this request. In the mean time why not check out some of the other opensource projects available, contributions are greatly appreciated!

Some of the most popular are


PR Statistics

#️⃣ PR Number: Line Additions: 🗑️ Line Deletions:
137 1427 285
📑 Files Changed: Repo Stars: 🔱 Total Forks:
30 27 79

Copy link

🙏 Thanks for your pull request @fullspeccoder, The team will now review and merge this request. In the mean time why not check out some of the other opensource projects available, contributions are greatly appreciated!

Some of the most popular are


PR Statistics

#️⃣ PR Number: Line Additions: 🗑️ Line Deletions:
137 1426 284
📑 Files Changed: Repo Stars: 🔱 Total Forks:
30 27 79

Copy link
Collaborator

@r4pt0s r4pt0s left a comment

Choose a reason for hiding this comment

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

🔥🔥🔥

@r4pt0s r4pt0s merged commit 5bf8296 into zero-to-mastery:main Oct 16, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants