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

🐛 [Firebase Data Connect] Add directive @default_on_updated or something similar #17075

Open
silentlie opened this issue Feb 11, 2025 · 3 comments
Labels
Needs Attention This issue needs maintainer attention. plugin: data_connect type: enhancement New feature or request

Comments

@silentlie
Copy link

silentlie commented Feb 11, 2025

What feature would you like to see?

Currently, there is no way to use an expr request.time on the server-side Timestamp on updated; the only way is to use the client-side Timestamp.
It would be nice to have something like @default_on_updated(expr: "request.time") or @default_on_updated(expr: "auth.uid").

@silentlie silentlie added Needs Attention This issue needs maintainer attention. type: enhancement New feature or request labels Feb 11, 2025
@SelaseKay
Copy link
Contributor

Hi @silentlie, based on my understanding, this documentation outlines how to achieve the feature you're requesting. Let me know if you need further clarification.

@SelaseKay SelaseKay added blocked: customer-response Waiting for customer response, e.g. more information was requested. and removed Needs Attention This issue needs maintainer attention. labels Feb 11, 2025
@silentlie
Copy link
Author

silentlie commented Feb 12, 2025

After rechecking the docs, I’m unsure how it works anymore. As I tested on my local machine, it can work as createdAt which is good, but updated does nothing. There should be a difference between createdAt and updatedAt.
Please give me an example if possible.

@google-oss-bot google-oss-bot added Needs Attention This issue needs maintainer attention. and removed blocked: customer-response Waiting for customer response, e.g. more information was requested. labels Feb 12, 2025
@SelaseKay
Copy link
Contributor

Hi @silentlie, I'm testing this locally. I'll provide feedback once I'm done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Attention This issue needs maintainer attention. plugin: data_connect type: enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants