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.

Sign up for GitHub

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

Jump to bottom

[Backend] Add missing test cases for TelegramUserDBConnector and UserDBConnector #414

Closed
djeck1432 opened this issue Dec 19, 2024 · 1 comment · Fixed by #419
Closed

[Backend] Add missing test cases for TelegramUserDBConnector and UserDBConnector #414

djeck1432 opened this issue Dec 19, 2024 · 1 comment · Fixed by #419
Assignees
@PavelShkurko
Labels
Backend good first issue Good for newcomers ODHack11 ONLY_FOR_NEW_CONTRIBUTORS

Comments

@djeck1432
Copy link
Owner

🚨 Attention: Please read the guidelines before applying; otherwise, you will be ignored. 🚫📋
0. Please, join the telegram group to keep in touch https://t.me/+m0IGRiBvLtsyMzIy

  1. Read guideline here
  2. Add test cases for TelegramUserDBConnector
  3. Add test UserDBConnector for next crud methods:
  • get_users_for_notifications,
  • get_contract_address_by_wallet_id,
  • create_user, update_user_contract,
  • fetch_user_history, delete_user_by_wallet_id
@PavelShkurko
Copy link
Contributor

PavelShkurko commented Dec 19, 2024 •
edited
Loading

I like strawberry. I’m interested in this one. May I try it

@djeck1432 djeck1432 linked a pull request Dec 20, 2024 that will close this issue
Feat/add-missing-test-cases #419
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@PavelShkurko PavelShkurko

Labels
Backend good first issue Good for newcomers ODHack11 ONLY_FOR_NEW_CONTRIBUTORS
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Feat/add-missing-test-cases PavelShkurko/spotnet
2 participants
@djeck1432 @PavelShkurko