Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add Support for upsert Semantics in save Method for PostgreSQL #3053

Closed
FWangZil opened this issue Oct 16, 2023 · 1 comment · Fixed by #3324
Closed

Add Support for upsert Semantics in save Method for PostgreSQL #3053

FWangZil opened this issue Oct 16, 2023 · 1 comment · Fixed by #3324
Assignees
Labels
enhancement help wanted planned This issue/proposal is planned into our next steps.

Comments

@FWangZil
Copy link

1. What version of Go and system type/arch are you using?

go version go1.20.9 darwin/arm64

2. What version of GoFrame are you using?

v2.5.4

3. Can this issue be re-produced with the latest release?

Yes.

4. What did you do?

I attempted to use the save method with PostgreSQL as the database.

5. What did you expect to see?

I expected the save method to support PostgreSQL, especially since PostgreSQL already supports the upsert operation. I suggest adding support for the upsert semantics. For reference, here's the official PostgreSQL documentation.

6. What did you see instead?

I received a message indicating that PostgreSQL is not supported by the save method.


@gqcn gqcn added enhancement help wanted planned This issue/proposal is planned into our next steps. labels Oct 19, 2023
@github-actions
Copy link

Hello @FWangZil. We like your proposal/feedback and would appreciate a contribution via a Pull Request by you or another community member. We thank you in advance for your contribution and are looking forward to reviewing it!
你好 @FWangZil。我们喜欢您的提案/反馈,并希望您或其他社区成员通过拉取请求做出贡献。我们提前感谢您的贡献,并期待对其进行审查。

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement help wanted planned This issue/proposal is planned into our next steps.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants