Skip to content
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

Paragon StatusAlert deprecation #574

Merged
merged 11 commits into from
Aug 11, 2022

Conversation

abdullahwaheed
Copy link
Contributor

Ticket

Migrate off deprecated Paragon components

What has changed

Removed deprecated StatusAlert component of paragon and updated it with Alert

References

Paragon StatusAlert
Paragon Alert

@abdullahwaheed abdullahwaheed requested a review from a team June 22, 2022 06:42
@abdullahwaheed abdullahwaheed self-assigned this Jun 22, 2022
@codecov
Copy link

codecov bot commented Jun 22, 2022

Codecov Report

Merging #574 (ef444cf) into master (392167e) will increase coverage by 4.23%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #574      +/-   ##
==========================================
+ Coverage   60.37%   64.61%   +4.23%     
==========================================
  Files          45       45              
  Lines         795      797       +2     
  Branches      146      148       +2     
==========================================
+ Hits          480      515      +35     
+ Misses        305      272      -33     
  Partials       10       10              
Impacted Files Coverage Δ
src/profile/AgeMessage.jsx 100.00% <ø> (+66.66%) ⬆️
src/profile/ProfilePage.jsx 82.19% <ø> (+2.73%) ⬆️
src/profile/forms/SocialLinks.jsx 98.43% <100.00%> (+45.21%) ⬆️
src/profile/data/selectors.js 91.42% <0.00%> (+0.95%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Copy link
Contributor

@davidjoy davidjoy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small change to one of the tests... otherwise I think this looks good.

@abdullahwaheed abdullahwaheed requested a review from davidjoy July 4, 2022 10:46
@davidjoy davidjoy merged commit 34a9caa into master Aug 11, 2022
@davidjoy davidjoy deleted the abdullah/paragon-status-alert-deprecations branch August 11, 2022 12:20
snglth pushed a commit to Abstract-Tech/community-theme-profile that referenced this pull request Jul 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants