Skip to content

Kick user if it has no username#30

Merged
Kavuti merged 1 commit intomasterfrom
feature/26-auto-kick-no-username
Nov 2, 2020
Merged

Kick user if it has no username#30
Kavuti merged 1 commit intomasterfrom
feature/26-auto-kick-no-username

Conversation

@iflare3g
Copy link
Copy Markdown
Collaborator

@iflare3g iflare3g commented Oct 2, 2020

No description provided.

@iflare3g iflare3g force-pushed the feature/26-auto-kick-no-username branch from 058251c to 00af70e Compare October 2, 2020 13:33
@iflare3g iflare3g changed the title - Kick user if it has no username Kick user if it has no username Oct 2, 2020
Comment thread tg_bot/modules/welcome.py Outdated
@Kavuti Kavuti linked an issue Oct 5, 2020 that may be closed by this pull request
@iflare3g iflare3g force-pushed the feature/26-auto-kick-no-username branch 2 times, most recently from 76af8a4 to b02d105 Compare October 9, 2020 14:33
@MattiaFailla MattiaFailla self-requested a review October 14, 2020 09:12
@MattiaFailla MattiaFailla requested a review from Kavuti October 14, 2020 09:14
@MattiaFailla
Copy link
Copy Markdown
Member

@Kavuti Could you please left a review so we can merge this PR?

Comment thread tg_bot/modules/welcome.py Outdated
message.reply_text(
"Diamine, non riesco a bannare questo utente."
)
chat.kick_member(user_id, until_date=1602201900)
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

until_date dovrebbe essere dinamico

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.

until_date prende in input tempo unix, dovrebbe essere calcolato nel seguente modo se vogliamo sbloccare dopo cinque minuti:

until_date=time.time() + 300

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

forse non ho pushato, devo ricontrollare

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Aggiornato!

Copy link
Copy Markdown
Collaborator

@Kavuti Kavuti left a comment

Choose a reason for hiding this comment

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

Bisogna modificare la until_date del kick che attualmente è hard-coded ad una data precisa

- Debugging welcome.py
- Improving username check
- Improving username check by checking the existence of the `username` attribute. Refactorizing username check by moving the functionality to the main new_user loop
- Fix kick member if username is not set
@iflare3g iflare3g force-pushed the feature/26-auto-kick-no-username branch from b02d105 to 80dc0b9 Compare November 2, 2020 17:45
@iflare3g
Copy link
Copy Markdown
Collaborator Author

iflare3g commented Nov 2, 2020

Adesso dovrebbe essere corretto @Kavuti , @MattiaFailla

Copy link
Copy Markdown
Collaborator

@Kavuti Kavuti left a comment

Choose a reason for hiding this comment

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

Ottimo lavoro

@Kavuti Kavuti merged commit bc98972 into master Nov 2, 2020
@MattiaFailla MattiaFailla deleted the feature/26-auto-kick-no-username branch August 12, 2021 11:22
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.

Identificazione e kick degli utenti scammer di criptovalute

3 participants