Skip to content

Conversation

@helloiamlukas
Copy link
Contributor

fixes #67

@edalzell
Copy link
Collaborator

@helloiamlukas can you please resolve the merge issues?

@what-the-diff
Copy link

what-the-diff bot commented Nov 10, 2022

  • The id attribute is now optional when creating a new entry.
  • A bug was fixed where the Entry model would not be found if it did not have an ID set yet, but had other attributes that could be used to find or create the record (e.g., origin_id).

@jasonvarga
Copy link
Member

FYI currently this reverts the fix from #19. If you were to import entries, they'd lose their existing IDs.

I'll update this PR appropriately.

@jasonvarga jasonvarga changed the title remove id from fill() to prevent postgres error Only fill entry model with id if it already has one, to prevent postgres error Nov 11, 2022
@jasonvarga jasonvarga merged commit fd7120c into statamic:master Nov 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Postgres: Can't create entry

3 participants