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

UpdatedAt is not set in the sample server with Azure SQL #210

Closed
SorenBech2 opened this issue Jan 21, 2025 · 1 comment · Fixed by #218
Closed

UpdatedAt is not set in the sample server with Azure SQL #210

SorenBech2 opened this issue Jan 21, 2025 · 1 comment · Fixed by #218
Labels
Milestone

Comments

@SorenBech2
Copy link

In the server/client sample the UpdateAt is not used. It is defined as nullable on the server side and not generated when a row is written to the table. The solution "works" since the null value is syncronized back and forth so as such not a bug. However I already have a database (Azure Mobile App) were the UpdatedAt field holds data and I'm struggling with converting this to CommunityToolkit.DataSync (Azure Sql on the server side and SQLite on the client side) as I haven't succeded in developing the correct conversion.

I have tried a host of options - to many to list here. It would therefore be great to have a working sample where the UpdatedAt is implemented - which I assume is necessary to manage the syncronization properly?

Appricitate your help in this.

@SorenBech2 SorenBech2 added the Requires Triage This issue has not been checked by the project team. label Jan 21, 2025
@adrianhall adrianhall added Examples and removed Requires Triage This issue has not been checked by the project team. labels Jan 21, 2025
@adrianhall adrianhall added this to the 9.0.0 milestone Jan 21, 2025
@adrianhall adrianhall changed the title UpdateAt is not used in samples UpdatedAt is not set in samples Jan 21, 2025
@adrianhall adrianhall changed the title UpdatedAt is not set in samples UpdatedAt is not set in the sample server with Azure SQL Jan 21, 2025
@adrianhall adrianhall linked a pull request Jan 21, 2025 that will close this issue
@adrianhall
Copy link
Collaborator

There was indeed a bug in the server. I did not replicate the AzureSQL requirements for database triggers in the server. That is being done within prs/218

In the same PR, I've also added a test suite for testing the sample server, with the "UpdatedAt" create property test being the first in the set. I suspect that this set will continue to grow in the future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants