Modifications to battle logs summary and telegram notification#12
Open
virgaux wants to merge 228 commits intoPCJones:masterfrom
Open
Modifications to battle logs summary and telegram notification#12virgaux wants to merge 228 commits intoPCJones:masterfrom
virgaux wants to merge 228 commits intoPCJones:masterfrom
Conversation
*Updated battle summary log *added telegram notification Additional installation for telegram notif : npm install --save telegram-notify config: token - your telegram bot token, create bot: https://t.me/BotFather chatId - telegram chat id for notify, get the id: https://t.me/get_id_bot
* Update much accurate ERC rating * Fix some logs for better battle summary * Update New Feature in README * Add installation guide for Telegram Notification in README
Owner
|
I'll modify the telegram part so that nobody has to create a bot, that might take some time. |
Contributor
Author
|
There is a public token that can be used that I made. This can be used by anyone. It needs more modification to be fancy but it works. The token is updated on my fork and just look for @Battle_Result_Bot |
*Add npm install for telegram and chromium installation in intall.bat * fix some font issue. * update README
* Update the quest battle log to have accurate info for quest reward. * Quest reward can be redeemed once available after battle. * add automation for Install.bat to run stat.bat after installation.
Fixed the updated Quest status error
- fixed API unable to recognize quest priority - fixed rented card not being used. (Optional to download) - Update Install.bat
virgaux
commented
Sep 26, 2021
Contributor
Author
virgaux
left a comment
virgaux
left a comment
There was a problem hiding this comment.
- API unable to recognize quest priority is still not fixed.
hopefully quest priority is now working cuz I give up. XD
- Accurate battle summary. - Can identify if it is draw - Claim reward function after battle if available. -added alert notification to telegram
- fix undefined error when selecting cards if api was not used.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hi Jones,
I just made some modifications to the battle logs summary. Also fixed the ERC to be more accurate and include the current and new rating after the battle.
I added telegram notification if ever the user wants it. This is what it looks like.