-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
## Description The embed is already filtering these out and replacing them with its own content. I'd say PEP shouldn't be showing them at all, since they're not actionable. We do still need to keep them in the v0 API, to avoid disrupting the legacy embed and v0 API clients. The boolean flag is inelegant, but it will be removed along with the v0 API. ## Test Plan `yarn test` with new/updated tests.
- Loading branch information
Showing
5 changed files
with
35 additions
and
391 deletions.
There are no files selected for viewing
This file contains 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
This file contains 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
This file contains 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
Oops, something went wrong.