-
Notifications
You must be signed in to change notification settings - Fork 66
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
docs: update documentation for deprecated components #2479
Conversation
✅ Deploy Preview for paragon-openedx ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## master #2479 +/- ##
=======================================
Coverage 91.70% 91.70%
=======================================
Files 236 236
Lines 4219 4219
Branches 1021 1021
=======================================
Hits 3869 3869
Misses 346 346
Partials 4 4 ☔ View full report in Codecov by Sentry. |
Thanks for the pull request, @viktorrusakov! Please note that it may take us up to several weeks or months to complete a review and merge your PR. Feel free to add as much of the following information to the ticket as you can:
All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here. Please let us know once your PR is ready for our review and all tests are green. |
Hi @viktorrusakov! Please let me know if you have any questions regarding submitting your CLA form. If you are contributing as part of another organization, you'll need to be added to their entity CLA. |
Hi @mphilbrick211, Viktor was previously under Raccoon Gang's CLA but he is no longer with Raccoon Gang. That said, he is still a Core Contributor and we believe he has signed an individual CLA form already. Perhaps he is falling into this camp now:
@viktorrusakov When you get a chance, do you mind filling out the individual CLA form again? |
Thanks, @adamstankiewicz, I appreciate the clarification, and apologies @viktorrusakov for not remembering you're a CC! If you don't mind re-submitting the form, that would be great! If there's any issue, I can look into it for you. |
Hi @adamstankiewicz, @mphilbrick211! Thanks for the clarification, I totally forgot that there was something wrong with my CLA check. I've resubmitted the form just now, let me now if anything else is required from my side. |
Hi @viktorrusakov! Did you get confirmation of your CLA going through? If not, I can check on it for you. |
a33047c
to
df746d8
Compare
Hi @mphilbrick211, no, I don't think so, I didn't receive any feedback after submitting the form and CLA check still fails after re-running it. I've resubmitted the form just now just in case I made a typo somewhere last week. |
Great, thanks, @viktorrusakov! I will check on Monday to make sure it's been processed. |
Hi @viktorrusakov! I checked on your CLA status, and it has been received. Tomorrow, you should be able to rebase and your CLA check should turn green. |
eed4ec7
to
162f425
Compare
Thank you @mphilbrick211, it works now! |
@viktorrusakov 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future. |
🎉 This PR is included in version 21.1.3 🎉 The release is available on: Your semantic-release bot 📦🚀 |
🎉 This PR is included in version 22.0.0-alpha.6 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Description
Updates documentation for deprecated components to give more accurate replacement options.
Deploy Preview
Include a direct link to your changes in this PR's deploy preview here (e.g., a specific component page).
Merge Checklist
example
app?wittjeff
andadamstankiewicz
as reviewers on this PR.Post-merge Checklist