Skip to content

feat(database): support for postgres#3998

Merged
livio-a merged 86 commits intomainfrom
postgres
Aug 31, 2022
Merged

feat(database): support for postgres#3998
livio-a merged 86 commits intomainfrom
postgres

Conversation

@adlerhurst
Copy link
Copy Markdown
Member

@adlerhurst adlerhurst commented Jul 19, 2022

No description provided.

@adlerhurst adlerhurst self-assigned this Jul 19, 2022
@netlify
Copy link
Copy Markdown

netlify Bot commented Jul 19, 2022 •
edited
Loading

✅ Deploy Preview for docs-zitadel-com ready!

Name Link
🔨 Latest commit 766f1cd
🔍 Latest deploy log https://app.netlify.com/sites/docs-zitadel-com/deploys/630f0fcb41ce9c00089f17a7
😎 Deploy Preview https://deploy-preview-3998--docs-zitadel-com.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

This was linked to issues Jul 19, 2022
[v2] Change to database config #3600
Closed
Support Postgresql #3898
Closed
@fforootd fforootd changed the title feat(database): add support for postgres feat(database): support for postgres Jul 19, 2022
@adlerhurst adlerhurst requested a review from livio-a July 25, 2022 15:22
@adlerhurst adlerhurst assigned livio-a and unassigned adlerhurst Jul 25, 2022
@codecov
Copy link
Copy Markdown

codecov Bot commented Jul 25, 2022 •
edited
Loading

Codecov Report

Merging #3998 (766f1cd) into main (4c26665) will decrease coverage by 0.04%.
The diff coverage is 54.48%.

@@            Coverage Diff             @@
##             main    #3998      +/-   ##
==========================================
- Coverage   47.10%   47.05%   -0.05%     
==========================================
  Files         622      622              
  Lines       55073    55150      +77     
==========================================
+ Hits        25941    25951      +10     
- Misses      27854    27906      +52     
- Partials     1278     1293      +15
Impacted Files Coverage Δ
cmd/initialise/verify_zitadel.go 29.41% <0.00%> (-3.93%) ⬇️
internal/api/grpc/admin/import.go 0.00% <0.00%> (ø)
internal/api/grpc/management/actions_converter.go 0.00% <0.00%> (ø)
internal/command/user_human_otp.go 27.89% <0.00%> (-0.59%) ⬇️
internal/command/user_idp_link.go 69.60% <0.00%> (-2.11%) ⬇️
internal/eventstore/handler/crdb/failed_stmt.go 50.00% <ø> (ø)
internal/eventstore/handler/crdb/init.go 2.03% <0.00%> (-0.01%) ⬇️
internal/eventstore/repository/sql/query.go 82.06% <ø> (-0.25%) ⬇️
...al/eventstore/v1/internal/repository/sql/config.go 0.00% <ø> (ø)
...nal/eventstore/v1/internal/repository/sql/query.go 86.87% <ø> (-0.17%) ⬇️
... and 73 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@adlerhurst adlerhurst marked this pull request as ready for review July 25, 2022 15:31
@livio-a
Copy link
Copy Markdown
Member

livio-a commented Aug 26, 2022

current state: @adlerhurst checks if "AS OF SYSTEM TIME" for the queries (esp. for the current sequences) is necessary

adlerhurst and others added 24 commits August 26, 2022 10:37
@adlerhurst
Merge branch 'main' into postgres
7897784
@adlerhurst
prerelease
0bca88d
@adlerhurst
Merge branch 'main' into postgres
4855de3
@adlerhurst
fix: add sequence to v1 "reduce paniced"
4e45ab9
@adlerhurst
log if nil
3937094
@adlerhurst
add logging
0a2b765
@adlerhurst
fix: log output
b6984fe
@adlerhurst
fix(import): check if org exists and user
cf9b00a
@adlerhurst
refactor: imports
e4d1c82
@adlerhurst
fix(user): ignore malformed events
11736c1
@adlerhurst
Merge branch 'fix-import-check-exists' into postgres
ce96586
@adlerhurst
refactor: method naming
922802e
@adlerhurst
fix: test
2adeee6
@adlerhurst
refactor: correct errors.Is call
04ba397
@adlerhurst
Merge branch 'fix-import-check-exists' into postgres
4255d5c
@eliobischof
ci: don't build dev binaries on main
29fa507
@adlerhurst
Merge remote-tracking branch 'origin/omit-debug-build' into postgres
cd58ee8
@adlerhurst
fix(go releaser): update version to 1.11.0
2933568
@adlerhurst
fix(user): projection should not break
3babd2e
@adlerhurst
fix(user): handle error properly
93d4f7d
@adlerhurst
Merge branch 'main' into postgres
186846d
@adlerhurst
docs: correct config example
3599d73
@livio-a
Update .releaserc.js
d319845
@livio-a
Update .releaserc.js
a7733e7
@livio-a livio-a enabled auto-merge (squash) August 31, 2022 07:37
@livio-a livio-a merged commit 77b4fc5 into main Aug 31, 2022
@livio-a livio-a deleted the postgres branch August 31, 2022 07:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@livio-a livio-a livio-a approved these changes

+1 more reviewer

@eliobischof eliobischof eliobischof left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@livio-a livio-a

Labels

backend docs Improvements or additions to documentation enhancement New feature or request gomod lang: go Pull requests that update Go code

Projects

No open projects
Kanban
Archived in project

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Support Postgresql [v2] Change to database config

4 participants

@adlerhurst @fforootd @livio-a @eliobischof