AppFlowy icon indicating copy to clipboard operation
AppFlowy copied to clipboard

refactor: create row workflow

Open richardshiue opened this issue 11 months ago • 0 comments

requires https://github.com/AppFlowy-IO/AppFlowy/pull/4688

  • Move some fragmented create row logic from event handler to database editor
  • Streamline row duplication logic
  • Remove unnecessary Arc<Field>. The filter, sort and group controllers already are Arcs, and in the instances where we use the struct field, we are just referencing it. So we probably don't need to put it in an Arc.

Feature Preview


PR Checklist

  • [x] My code adheres to AppFlowy's Conventions
  • [ ] I've listed at least one issue that this PR fixes in the description above.
  • [x] I've added a test(s) to validate changes in this PR, or this PR only contains semantic changes.
  • [ ] All existing tests are passing.

richardshiue avatar Mar 04 '24 01:03 richardshiue