Skip to content

Feat/move max enchant to api - #94

Open
cataladev wants to merge 7 commits into
SkyCryptWebsite:devfrom
cataladev:feat/move-max-enchant-to-api
Open

cataladev wants to merge 7 commits into
SkyCryptWebsite:devfrom
cataladev:feat/move-max-enchant-to-api

Conversation

@cataladev

@cataladev cataladev commented Sep 13, 2026

Copy link
Copy Markdown

Moved enchants const to api from frontend 💀

SkyCryptWebsite/SkyCrypt-Frontend#345

@cataladev

Copy link
Copy Markdown
Author

had to clean up bullshit

@DuckySoLucky DuckySoLucky left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm, didn't test (will test tmrw)

@Zalk0 Zalk0 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Didn't test code but checked enchants levels quickly and found 2 that need updating

Comment thread src/constants/enchantments.go Outdated
Comment thread src/constants/enchantments.go Outdated
cataladev and others added 3 commits September 13, 2026 15:26
Co-authored-by: Zalko <88582103+Zalk0@users.noreply.github.com>
Co-authored-by: Zalko <88582103+Zalk0@users.noreply.github.com>
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