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

ModelInstanceDocument doc.replace, there are some fields that are not updated in db table #2499

Open
lyd0 opened this issue Oct 18, 2022 · 2 comments

Comments

@lyd0
Copy link

lyd0 commented Oct 18, 2022

Describe the bug

    const doc = await ModelInstanceDocument.create(ceramic, CONTENT0, midMetadata)
    await doc.replace(CONTENT1)

but in postgress ,the db table only updates 2 fields(last_anchored_at&updated_at) when doc updated, CONTENT1 data is not updated to table,maybe other fields should be updated as well

https://github.com/ceramicnetwork/js-ceramic/blame/33e3c0969c0161d5dc17b55501775385241066be/packages/core/src/indexing/postgres/postgres-index-api.ts#L57

Screenshots
If applicable, add screenshots to help explain your problem.
image

@lyd0
Copy link
Author

lyd0 commented Oct 18, 2022

@willex hello~ Looking forward to your reply

@stbrody
Copy link
Contributor

stbrody commented Oct 19, 2022

@lyd0 this does indeed look like a bug. Thanks for bringing it to our attention! We'll triage it at our next triage meeting in a week or so

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

No branches or pull requests

2 participants