Changes between Initial Version and Version 1 of Ticket #49532, comment 16
- Timestamp:
- 09/30/2022 03:09:18 PM (3 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #49532, comment 16
initial v1 1 1 [attachment:"49532.2.diff"] refreshes the patch against trunk and cleans up things a bit further. 2 2 3 @mukesh27 can you test again and confirm the reporter's issue is resolved? 3 @mukesh27 can you test again and confirm the reporter's issue is resolved? I think we can add a unit test here to verify the new behavior vs the old (maybe check the post id for the user's autosave doesn't change), let me know if you want to work on that. 4 4 5 5 @desrosj or @azaozz - any feedback on this change, anything I might be missing? I don't see a valid reason to keep deleting and recreating the autosave for the user when the post content matches; this change move the check earlier and returns early if the autosave being sent matches the post content.