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

feat: add Giveaway param in exemptmember #452

Merged
merged 5 commits into from
Apr 16, 2022

Conversation

imranbarbhuiya
Copy link
Contributor

Changes

This pr adds Giveaway param in exemptMembers function.

Ik we've new Function(), where we can access this, but when using new Function() all our javascript code becomes a string and that's why our ide/typescript can't show errors. Also, everyone doesn't know how to make new Function() async.

Status

  • These changes have been tested and formatted properly.
  • This PR includes only documentation changes (JSDoc, README or typings), no code change.
  • This PR introduces some breaking changes.

typings/index.d.ts Outdated Show resolved Hide resolved
@Nico105
Copy link
Collaborator

Nico105 commented Apr 16, 2022

btw for consistency you could also add the param for bonusEntry functions, if you want.

@imranbarbhuiya
Copy link
Contributor Author

imranbarbhuiya commented Apr 16, 2022

btw for consistency you could also add the param for bonusEntry functions, if you want.

I haven't checked any bonus entry-related codes yet so idk how it works and I don't have time to check all the bonus entries related codes. If anyone else is interested then go-ahead

@Nico105
Copy link
Collaborator

Nico105 commented Apr 16, 2022

btw for consistency you could also add the param for bonusEntry functions, if you want.

I haven't checked any bonus entry-related codes yet so idk how it works and I don't have time to check all the bonus entries related codes. If anyone else is interested then go-ahead

np. i will do it when i have time then.

@Nico105 Nico105 merged commit c6384ef into Androz2091:develop Apr 16, 2022
@imranbarbhuiya imranbarbhuiya deleted the develop branch April 16, 2022 15:58
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.

2 participants