-
Notifications
You must be signed in to change notification settings - Fork 27
Microsoft Teams Adaptive Cards #35
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Microsoft Teams Adaptive Cards #35
Conversation
Thanks for your contribution, I'm willing to merge this, but before we need to consider two more things:
Thanks again for your effort, because of people like you we can keep open-source alive 🙇 |
->notify(new SubscriptionCreated()); | ||
``` | ||
|
||
### Available Adaptive Card methods |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please resolve the merge conflict here, thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This merge conflict has now been resolved
@Tob0t As it is a breaking change, should we take the opportunity to only support PHP >8.1 or 8.2 |
@Tob0t Update notes and instructions have been added |
Hi @Tob0t, Is there a schedule to merge this PR? We’re looking forward to starting with Adaptive Cards and contributing more interesting features. |
Hi @nzwz, |
Two questions:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your effort 👍
Yes, let's go for >= 8.1 and then we can use Enums, feel free to create a follow up PR |
This PR is an update to use Adaptive Cards as proposed in issue
#31