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
fix: Clear fetched field when link field is cleared #15343
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.
@sumaiya2908 the fix does not work with this change #15320
You might have to update the code
@Mergifyio backport version-13-hotfix |
🟠 Waiting for conditions to match
|
Codecov Report
@@ Coverage Diff @@
## develop #15343 +/- ##
===========================================
+ Coverage 49.78% 50.80% +1.01%
===========================================
Files 745 742 -3
Lines 65234 65320 +86
Branches 5419 5432 +13
===========================================
+ Hits 32479 33186 +707
+ Misses 28825 28272 -553
+ Partials 3930 3862 -68
Flags with carried forward coverage won't be shown. Click here to find out more. |
❌ No backport have been created
|
@sumaiya2908 A support ticket was raised for Frappe v14 (develop) even though we haven't started stable releases for it? |
@gavindsouza the issue was raised for v13. And when I checked, it existed in develop too. So I raised a PR for develop, and then backported it. |
Issue
When the link field is cleared, the docfield which is fetched from the link field remains the same as the previous value and gets saved.
Before
the 'Name' field is read-only.
Screen.Recording.2021-12-17.at.9.17.18.AM.mov
After
Screen.Recording.2021-12-17.at.9.18.22.AM.mov
Steps to reproduce
a. Expected Result: the field which is being fetched, should be cleared.
b. Actual Result: it remains the same as the previous value and gets saved.