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

Property Views #732

Merged
merged 40 commits into from
May 20, 2024
Merged

Property Views #732

merged 40 commits into from
May 20, 2024

Conversation

FoseFx
Copy link
Member

@FoseFx FoseFx commented Apr 12, 2024

Closes #647
#646

@FoseFx FoseFx force-pushed the issue/646-property-view-aggregate branch from c5232e7 to ea1daa4 Compare April 16, 2024 18:05
@FoseFx FoseFx force-pushed the issue/646-property-view-aggregate branch from 33b36a7 to df41811 Compare April 24, 2024 10:01
@FoseFx FoseFx force-pushed the issue/646-property-view-aggregate branch from e93e9d3 to 573bab8 Compare May 2, 2024 13:32
@FoseFx FoseFx self-assigned this May 12, 2024
@FoseFx FoseFx marked this pull request as ready for review May 12, 2024 19:06
@FoseFx FoseFx requested a review from a team as a code owner May 12, 2024 19:06
@FoseFx FoseFx requested review from MaxSchaefer and PaulKalho and removed request for a team May 12, 2024 19:06
FoseFx and others added 9 commits May 18, 2024 15:26
…ate_property_view_rule.go

Co-authored-by: Paul Kalhorn <67231903+PaulKalho@users.noreply.github.com>
…k_views_postgres/task_views_postgres.go

Co-authored-by: Paul Kalhorn <67231903+PaulKalho@users.noreply.github.com>
…k_views_postgres/task_views_postgres.go

Co-authored-by: Paul Kalhorn <67231903+PaulKalho@users.noreply.github.com>
Co-authored-by: Paul Kalhorn <67231903+PaulKalho@users.noreply.github.com>
Copy link
Contributor

@PaulKalho PaulKalho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good now, we should still take care of this one thing in the projection.

@FoseFx FoseFx merged commit 8628d8b into main May 20, 2024
40 checks passed
@FoseFx FoseFx deleted the issue/646-property-view-aggregate branch May 20, 2024 13:57
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.

CRUD PropertyView
2 participants