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

fixed amari req #84

Merged
merged 1 commit into from Dec 31, 2021
Merged

fixed amari req #84

merged 1 commit into from Dec 31, 2021

Conversation

Lightninbolt986
Copy link
Contributor

@Lightninbolt986 Lightninbolt986 commented Dec 31, 2021

Summary

Give an overview
Amari reqs were a bit weird fixed that

Extra details

Give extra details to describe the changes
wamari req wasnt shown in embed and neither amari level nor wamari req not being met stopped user from entering so fixed that.

References

Text

Related Issue

Issue Name

Checks

  • Does the bot package work as intended?

@Assassin-1234 Assassin-1234 added bug Something isn't working good first issue Good for newcomers labels Dec 31, 2021
@Assassin-1234
Copy link
Member

how does this fixes the embeds? from what I can see, you can just put level and weeklyamari in your requirements to fix the issue instead of renaming the vars in the package itself

@Lightninbolt986
Copy link
Contributor Author

Lightninbolt986 commented Dec 31, 2021

how does this fixes the embeds? from what I can see, you can just put level and weeklyamari in your requirements to fix the issue instead of renaming the vars in the package itself

Cause it was put differently in the src in different locations. I made it amarilevel and amariweekly uniformly across all files. Making amarilevel into level would break the embed and not show the amari level req. If i use weeklyamari instead of amariweekly that would also mess stuff up

@Assassin-1234
Copy link
Member

alright thank you so much :) merging

@Assassin-1234 Assassin-1234 merged commit 529d0ce into Nuggies-bot:main Dec 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants