Skip to content

fixing upgrades#34

Open
jacksonc29 wants to merge 3 commits intorandomusert:mainfrom
jacksonc29:feature-add-achivment
Open

fixing upgrades#34
jacksonc29 wants to merge 3 commits intorandomusert:mainfrom
jacksonc29:feature-add-achivment

Conversation

@jacksonc29
Copy link

@jacksonc29 jacksonc29 commented Jan 15, 2026 •
edited
Loading

when testing the game i noticed both of base upgrades did not work. i will fix so when clicked it makes changes to the game ex. add curser.

@vercel
Copy link

vercel bot commented Jan 15, 2026

@jacksonc29 is attempting to deploy a commit to the randomusertgh's projects Team on Vercel.

A member of the Team first needs to authorize it.

@jacksonc29 jacksonc29 changed the title fixing upgrades so when clicked it makes changes to the game ex. adds clicker fixing upgrades Jan 20, 2026
@randomusert randomusert self-assigned this Jan 29, 2026
@randomusert randomusert added the bug Something isn't working label Jan 29, 2026
@vercel
Copy link

vercel bot commented Jan 29, 2026 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
moneyclick Ready Ready Preview, Comment Jan 29, 2026 4:58pm

randomusert
randomusert requested changes Jan 29, 2026
View reviewed changes
Copy link
Owner

@randomusert randomusert left a comment

Choose a reason for hiding this comment

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

it was inteonally that cheap.

randomusert
randomusert approved these changes Jan 29, 2026
View reviewed changes
Copy link
Owner

@randomusert randomusert left a comment

Choose a reason for hiding this comment

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

sure......

randomusert
randomusert requested changes Jan 29, 2026
View reviewed changes
Copy link
Owner

@randomusert randomusert left a comment

Choose a reason for hiding this comment

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

js is fine. The button changes in HTML though isn't. please note that the project uses tailwind css

@randomusert
Copy link
Owner

randomusert commented Jan 29, 2026

Before addressing these requests this PR will not be merged to prod. Please Note that as i use snyk for security vuln patching this pr might get blocked with issues for merging.

@randomusert
Copy link
Owner

randomusert commented Jan 29, 2026

Before addressing these requests this PR will not be merged to prod. Please Note that as i use snyk for security vuln patching this pr might get blocked with issues for merging.

Also the PR #32 will be blocked if i go trought with this PR also please consider contributing trough the dev repo instead

the dev repo: https://github.com/tinteeam/Moneyclick

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@randomusert randomusert randomusert requested changes

Assignees

@randomusert randomusert

Labels

bug Something isn't working

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jacksonc29 @randomusert