techlift-tech TelegramBotForPokerAnalysis issues

techlift-tech / TelegramBotForPokerAnalysis

0 stars 0 forks source link

issues

Newest

Newest Most commented Recently updated Oldest Least commented Least recently updated

create new commands

#28 disha-2401 closed 3 years ago
4
Add deploy.xml file to make the deploy automatic using github-action

#27 palashjhabak opened 3 years ago
4
Change all "var" to explicit variable type declaration like string or int or whatever

#26 palashjhabak opened 3 years ago
0
Faulty logic. you are iterating over all the commands and registering botcommand multiple times

#25 palashjhabak opened 3 years ago
0
use foreach instead of for as it provides much better readability of the code

#24 palashjhabak opened 3 years ago
0
await all async function calls

#23 palashjhabak opened 3 years ago
0
make it private readonly and move it above class constructors

#22 palashjhabak opened 3 years ago
0
Property should be private readonly and should be instantiated in the class constructor

#21 palashjhabak opened 3 years ago
0
Property does not need to be a public property

#20 palashjhabak opened 3 years ago
0
Remove Console.writeline

#19 palashjhabak opened 3 years ago
0
Make this method async and await the GetMeAsync method call

#18 palashjhabak opened 3 years ago
0
Remove this Get method, we done need a GET API in this controller

#17 palashjhabak opened 3 years ago
0
TelegramBot name as the class name is not descriptive enough. TelegramBotWebhookController or something like that will be better

#16 palashjhabak opened 3 years ago
0
Register commands with Telegram if its not already registered. #7

#10 palashjhabak opened 3 years ago
5
Get registered commands for from telegram #7

#9 palashjhabak opened 3 years ago
4
Outgoing Messages and Flow

#8 palashjhabak opened 3 years ago
7
Incoming Messages Keywords and Flow

#7 palashjhabak opened 3 years ago
14
fixes issue #4

#6 disha-2401 opened 3 years ago
2
Can we have a better name than NewMessage?

#5 palashjhabak opened 3 years ago
5
Move API token to appsettings.json and ready it via Dependency Injection

#4 palashjhabak opened 3 years ago
0
change Controller class name to PascalCase

#3 palashjhabak opened 3 years ago
0
Process incoming Webhook Data

#2 palashjhabak opened 3 years ago
8
Web Server to Process Webhook incoming requests from Telegram Server

#1 palashjhabak opened 3 years ago
4