Skip to content

feat(sidequest): add roasted apples sidequest#2086

Open
Eobam wants to merge 35 commits intohackclub:mainfrom
Eobam:main
Open

feat(sidequest): add roasted apples sidequest#2086
Eobam wants to merge 35 commits intohackclub:mainfrom
Eobam:main

Conversation

@Eobam
Copy link
Copy Markdown

@Eobam Eobam commented Apr 8, 2026

This PR adds the Roasted Apples sidequest. It adds the description, as well as submission flow! image of preview coming here soon

@Eobam Eobam marked this pull request as draft April 8, 2026 12:51
@Eobam Eobam marked this pull request as ready for review April 8, 2026 20:48
@Eobam Eobam changed the title Add Roasted Apples Sidequest feat: Add Roasted Apples Sidequest Apr 8, 2026
@daaminashai
Copy link
Copy Markdown
Member

Hello, please make sure your PR title follows https://www.conventionalcommits.org/en/v1.0.0/

Eg: feat(sidequest): added roasted apples sidequest

@Eobam Eobam changed the title feat: Add Roasted Apples Sidequest feat(sidequest): Add roasted apples sidequest Apr 9, 2026
@Eobam Eobam changed the title feat(sidequest): Add roasted apples sidequest feat(sidequest): added roasted apples sidequest Apr 9, 2026
@Eobam Eobam changed the title feat(sidequest): added roasted apples sidequest feat(sidequest): add roasted apples sidequest Apr 9, 2026
Comment thread db/seeds.rb Outdated
@Eobam Eobam marked this pull request as draft April 10, 2026 11:16
@abinnovator
Copy link
Copy Markdown

Approve it!!

@Eobam Eobam marked this pull request as ready for review April 10, 2026 17:50
@penguinencounter
Copy link
Copy Markdown
Contributor

ps: you might have to rebase this or merge main in to get the db check to pass

@abinnovator
Copy link
Copy Markdown

pls approve :pls:

Comment thread db/seeds.rb Outdated
Comment thread db/seeds.rb Outdated
Comment thread app/views/sidequests/show_roasted_apples.html.erb Outdated
@cskartikey
Copy link
Copy Markdown
Member

cc: @transcental do we want to gate apple dev license behind this sidequest?

@Eobam
Copy link
Copy Markdown
Author

Eobam commented Apr 14, 2026

we don't have to, but it currently isn't even in the shop

@transcental
Copy link
Copy Markdown
Member

transcental commented Apr 14, 2026

I'm fine w/ the licence being gated, however no shop item additions/removals/changes should be hardcoded.

@BookCatKid
Copy link
Copy Markdown
Contributor

@Eobam You did not resolve merge conflict, you left the markers!

@Eobam
Copy link
Copy Markdown
Author

Eobam commented Apr 14, 2026

bleh, you are right

@abinnovator
Copy link
Copy Markdown

when?

@abinnovator
Copy link
Copy Markdown

whennn

@BookCatKid
Copy link
Copy Markdown
Contributor

Leaving this thread here where I pointed out a bunch of issues if anyone wants to follow along:
https://hackclub.slack.com/archives/C0ALBKY8TEX/p1776380076182759

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.

8 participants