Remove concept of types, supportedTypes, BasicCardType enum #62
Conversation
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.
Only one editorial suggestion; otherwise good to go.
12e005e
to
e02da1f
Compare
@rsolomakhin you probably saw, I sent a bunch of pull requests updating the tests. I hacked up your handler tests, so please make sure I didn't screw them up :) @zouhir, heads up on the Microsoft side. |
ACK |
👍 |
Oki, sent removal patch on the Gecko side... |
Call for consensus to remove sent today: |
I believe there are libraries that support Basic Card. We want to understand the impact of deprecating this feature on that deployed software. Perhaps the editors can reach consensus on appropriate browser behavior when they encounter this deprecated feature. Thanks! |
Libraries would be unaffected in terms of behavior - passing in |
Blocked on CFC until 15th of November. |
closes #59
The following tasks have been completed:
Implementation commitments:
Preview | Diff