djeck1432 / spotnet

8 stars 63 forks source link

[BACKEND] Add crud method and write test cases for endpoint #204

Closed djeck1432 closed 6 days ago

djeck1432 commented 1 week ago

🚨 Attention: Please read the guidelines before applying; otherwise, you will be ignored. 🚫📋

  1. Please, join the telegram group to keep in touch https://t.me/+m0IGRiBvLtsyMzIy
  2. Read guideline here
  3. Add new crud method to TelegramDBConnector with name: allow_notification which will change field is_allowed_notification to True in TelegramUser model
  4. Add this method to endpoint here
  5. Create test cases for subscribe_to_notification negative and positive cases
  6. Add test cases in this file
Benjtalkshow commented 1 week ago

I can work on this. Thanks

djeck1432 commented 1 week ago

@Benjtalkshow You can't work on it, because it's for new comers, you have already done your task before. so please let new people on-board to the project too.

bloomingpeach commented 1 week ago

Hi, can I work on this issue @djeck1432

djeck1432 commented 1 week ago

@bloomingpeach read guideline and reapply

bloomingpeach commented 1 week ago

May I work on the issue? The issue involves adding new crud method to update the TelegramUser entity. After implementing the changes, I will have to write e2e test for the related controller/endpoint.

Thanks for your consideration

djeck1432 commented 1 week ago

@bloomingpeach nope, sorry, if you can't read guideline properly, not sure that you can tackle this task Read again GUIDELINE

bloomingpeach commented 1 week ago

@Benjtalkshow nope, sorry, if you can't read guideline properly, not sure that you can tackle this task Read again GUIDELINE

Hi not sure if you are referring to me, I dont add the apple because I am not applying from OD directly, very sorry if I mistaken the guideline

mimisavage commented 1 week ago

I would like to work on this, I like 🍎.

djeck1432 commented 1 week ago

@Benjtalkshow nope, sorry, if you can't read guideline properly, not sure that you can tackle this task Read again GUIDELINE

Hi not sure if you are referring to me, I dont add the apple because I am not applying from OD directly, very sorry if I mistaken the guideline

Applications missing this phrase will not be considered, as it indicates the guidelines were not read.

djeck1432 commented 1 week ago

@mimisavage you were assigned, please make sure that upcoming 12 hours draft PR will be ready, if you face any issue, text me in DM in telegram

emarc99 commented 1 week ago

A new contributor, I like 🍎. Let me do this

bloomingpeach commented 1 week ago

@Benjtalkshow nope, sorry, if you can't read guideline properly, not sure that you can tackle this task Read again GUIDELINE

Hi not sure if you are referring to me, I dont add the apple because I am not applying from OD directly, very sorry if I mistaken the guideline

Applications missing this phrase will not be considered, as it indicates the guidelines were not read.

Sorry, I will be more attentive next time applying

djeck1432 commented 1 week ago

A new contributor, I like 🍎. Let me do this

new issue will be posted soon, follow us in telegram group to be up-to-dated https://t.me/+m0IGRiBvLtsyMzIy

mimisavage commented 1 week ago

@mimisavage you were assigned, please make sure that upcoming 12 hours draft PR will be ready, if you face any issue, text me in DM in telegram

Okay sir

Criste-ai commented 1 week ago

Am I allowed to work on this?

djeck1432 commented 1 week ago

@mimisavage Hi, any updates, issue?

djeck1432 commented 1 week ago

@mimisavage you will be unassign in an hour

mimisavage commented 1 week ago

Please, I'm making a PR now sir

djeck1432 commented 1 week ago

@mimisavage you have 30 mins, after that you will be unassign

mimisavage commented 1 week ago
image

I am trying to push @djeck1432

djeck1432 commented 1 week ago

@mimisavage text me in telegram