add [properties-be] - project metadata#614
Merged
danielkweon merged 22 commits intomainfrom Dec 15, 2025
Merged
Conversation
whutchinson98
requested changes
Dec 15, 2025
rust/cloud-storage/properties_db_client/src/project_metadata/get.rs
Outdated
Show resolved
Hide resolved
rust/cloud-storage/properties_db_client/src/project_metadata/get.rs
Outdated
Show resolved
Hide resolved
Add display name constants for email thread metadata properties: Subject, Thread Started, Last Received, Last Sent, Messages.
Add ThreadMetadata struct for email thread metadata properties: subject, thread_started, last_received, last_sent, message_count.
Add get_thread_metadata query using LATERAL join to efficiently fetch subject, thread_started, last_received, last_sent, and message_count.
Add create_metadata_property_number helper for numeric metadata (e.g., message count). Add documentation comments to all metadata helper functions.
Add function to build thread metadata properties: Subject, Thread Started, Last Received, Last Sent, and Messages count.
Add EntityType::Thread case to fetch thread metadata properties when include_metadata=true is requested.
Thread permission records (EmailThreadPermission/UserItemAccess) are lazily initialized when a thread is first accessed via DSS. For owned threads, this initialization was bypassed, causing properties_service permission checks to fail. Added fallback that checks thread ownership via email_links.macro_id when no permission records exist.
Remove redundant _optional and _null variants by making base functions accept Option<T> and handle nulls internally.
ad5ff42 to
65bcc28
Compare
Add NAME and PARENT constants for project metadata display names.
Add ProjectMetadata struct for project metadata properties.
Add get_project_metadata query to fetch project metadata from Project table.
Add get_project_metadata_properties function and wire up EntityType::Project in the properties endpoint. Returns Name, Owner, Created At, Last Updated, and Parent Project metadata.
6f59e6b to
b83b63f
Compare
Base automatically changed from
daniel/m-5179--add-properties-backend-metadata
to
main
December 15, 2025 19:08
whutchinson98
approved these changes
Dec 15, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
add [properties-be] - project metadata