Skip to content
This repository was archived by the owner on May 24, 2024. It is now read-only.

[terra-alert] Deprecate terra-alert #3369

Closed
3 of 4 tasks
mjhenkes opened this issue Feb 24, 2021 · 1 comment
Closed
3 of 4 tasks

[terra-alert] Deprecate terra-alert #3369

mjhenkes opened this issue Feb 24, 2021 · 1 comment

Comments

@mjhenkes
Copy link
Contributor

mjhenkes commented Feb 24, 2021

Feature Request

Description

terra-application provides a notification banner. That components is recommended over usage of this component.

  • update the repo readme and badge

  • update any of the component doc pages, using the notices component. In this case, the maintenance notice variant would be appropriate with similar phrasing as used for terra-application-navigation

  • update the component readme (using just straight markdown), update the badge
    adding an independent deprecation guide doc page. This doc should contain what deprecation means, why we've made the change and how consumers should move forward.

After PR

  • npm deprecate with a message (we should use the exact same message as what is in our notices component on the main doc pages so these match, plus also on the component readme.md - all can link to our deprecation guide doc).

Audit #3039

Additional Context / Screenshots

@ Mentions

@stale
Copy link

stale bot commented Oct 19, 2021

This issue has been automatically marked as inactive because it has not had recent activity. It will be closed in seven days if no further activity occurs. Thank you for your contributions.

@stale stale bot added the inactive label Oct 19, 2021
@stale stale bot closed this as completed Oct 26, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants