-
Notifications
You must be signed in to change notification settings - Fork 19
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
Read attributes from markdown for mentions #675
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.
Looks fine but could you add a couple tests for this use case ? 🙏
I'm trying to do this but don't really understand the code structure here. Where would this be tested? It's the greyed out code for this module that's confusing me |
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## main #675 +/- ##
============================================
+ Coverage 86.83% 89.77% +2.94%
============================================
Files 140 78 -62
Lines 16120 13495 -2625
Branches 750 0 -750
============================================
- Hits 13997 12115 -1882
+ Misses 1941 1380 -561
+ Partials 182 0 -182
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
Kudos, SonarCloud Quality Gate passed! |
When there was the refactor to add the ability to pass attributes in when creating links, this function was missed.
Amending this function to pass through (specified) attributes means that now we will be able to pass a mention from the plain text editor (ie "markdown") into the rich text editor and have the rust model be instantiated containing that mention with the required attributes.