Skip to content
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

Drop section moves if they are also deleted or inserted #577

Closed
wants to merge 5 commits into from

Conversation

rnystrom
Copy link
Contributor

@rnystrom rnystrom commented Mar 22, 2017

Changes in this pull request

Issue fixed: #430

Checklist

  • All tests pass. Demo project builds and runs.
  • I added tests, an experiment, or detailed why my change isn't tested.
  • I added an entry to the CHANGELOG.md for any breaking changes, enhancements, or bug fixes.

@facebook-github-bot
Copy link
Contributor

@rnystrom updated the pull request - view changes

@rnystrom rnystrom added this to the 3.0.0 milestone Mar 22, 2017
Copy link
Contributor

@jessesquires jessesquires left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🌮

@@ -77,9 +77,10 @@ - (instancetype)initWithInsertSections:(NSIndexSet *)insertSections
const NSInteger from = move.from;
const NSInteger to = move.to;

// if the move is already deleted or inserted, discard it and use delete+insert instead
// if the move is already deleted or inserted, discard it because count-changing operations must match
// with data source changesw
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

changesw typo

@facebook-github-bot
Copy link
Contributor

@rnystrom updated the pull request - view changes

@facebook-github-bot
Copy link
Contributor

@rnystrom has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@jessesquires jessesquires deleted the moves-deleted branch March 24, 2017 15:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Drop section moves when the section was deleted in IGListBatchUpdateData
4 participants