uptime-monitor icon indicating copy to clipboard operation
uptime-monitor copied to clipboard

ms teams notification added

Open byteab opened this issue 3 years ago • 5 comments

byteab avatar Aug 06 '22 04:08 byteab

CLA Assistant Lite bot: Thanks for your submission! We ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by adding a comment below using this text:


I have read the CLA Document and I hereby sign the CLA


You can retrigger this bot by commenting recheck in this Pull Request

upptime-bot avatar Aug 06 '22 04:08 upptime-bot

recheck

byteab avatar Aug 06 '22 06:08 byteab

@AnandChowdhary sorry for mentioning! do you have time to review this?

byteab avatar Aug 07 '22 04:08 byteab

Thanks for the PR! The diff is too big so I have trouble reviewing :( You don't have to modify the .js files since those are auto-generated. Can you please edit the branch with only changes to .ts source files? Thanks so much again for the PR!

AnandChowdhary avatar Aug 10 '22 16:08 AnandChowdhary

@AnandChowdhary thanks for your comment. those are build files, if any workflow in the library auto generate them after merging the PR, I can cherry-pick that built commit.

byteab avatar Aug 11 '22 16:08 byteab

⚠️ This PR has not seen any activity in the past 2 months so I'm marking it as stale. I'll close it if it doesn't see any activity in the coming week.

upptime-bot avatar Oct 11 '22 02:10 upptime-bot

Hey, yes, we only need to update the .ts files and the .js ones will auto generate via CI :)

AnandChowdhary avatar Oct 11 '22 06:10 AnandChowdhary

⚠️ This PR has not seen any activity in the past 2 months so I'm marking it as stale. I'll close it if it doesn't see any activity in the coming week.

upptime-bot avatar Dec 12 '22 01:12 upptime-bot