Skip to content
This repository has been archived by the owner. It is now read-only.

SequenceFeature doesn't set lastInsertValue correctly before the inse… #146

Closed
wants to merge 1 commit into from

Conversation

@alexsds
Copy link

alexsds commented Jul 6, 2016

…rt was done
Fix for this issue: #142
it is always +1 higher becouse lastInsertValue set from nextSequenceId()

@alexsds alexsds force-pushed the alexsds:issues/142 branch 2 times, most recently from 907a5d7 to dbc8534 Jul 6, 2016
Aleksandr Kovalenko
…rt was done
@alexsds alexsds force-pushed the alexsds:issues/142 branch from dbc8534 to 90a300b Jul 11, 2016
@alexsds

This comment has been minimized.

Copy link
Author

alexsds commented Aug 8, 2017

@EvanDotPro please review

@ezimuel ezimuel added the bug label Aug 6, 2018
@ezimuel

This comment has been minimized.

Copy link
Member

ezimuel commented Aug 6, 2018

We need also documentation on the Sequence feature, see #165

@ezimuel ezimuel added this to the 2.10.0 milestone Aug 6, 2018
@ezimuel ezimuel self-assigned this Aug 6, 2018
@michalbundyra michalbundyra modified the milestones: 2.10.0, 2.11.0 Nov 26, 2019
michalbundyra added a commit that referenced this pull request Dec 31, 2019
michalbundyra added a commit that referenced this pull request Dec 31, 2019
Close #146
Fix #142
@michalbundyra

This comment has been minimized.

Copy link
Member

michalbundyra commented Dec 31, 2019

Rebased, updated and merged in 53fcd5f.

Thanks, @alexsds!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.