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.

Sign up for GitHub

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

Jump to bottom

[chore] Standardize database queries, use bun.Ident() properly #886

Merged
merged 29 commits into from
Oct 8, 2022
Merged

[chore] Standardize database queries, use bun.Ident() properly #886

merged 29 commits into from
Oct 8, 2022

Conversation

tsmethurst
Copy link
Contributor

  • Use bun.Ident() appropriately to fix postgres issues.
  • Prefer using id to WherePK() (which sometimes doesn't work as expected on postgres).
  • Fix session db select.
  • Add delete account function.
  • Use TableExpr where possible to avoid instantiating pointers to empty structs.
  • Add more tests.

@tsmethurst
use bun.Ident for user queries
e84b62d
@tsmethurst
use bun.Ident for account queries
4c002ba
@tsmethurst
use bun.Ident for media queries
5f345a2
@tsmethurst
add DeleteAccount func
70edb11
@tsmethurst
remove CaseInsensitive in Where+use Ident ipv Safe
fa88a90
@tsmethurst
update admin db
6227de7
@tsmethurst
update domain, use ident
7c0d44a
@tsmethurst
update emoji, use ident
fc078dc
@tsmethurst
update instance queries, use bun.Ident
79ee041
@tsmethurst
fix media
18b1b05
@tsmethurst
update mentions, use bun ident
e48c3d0
@tsmethurst
update relationship + tests
481c7ea
@tsmethurst
use tableexpr
916b652
@tsmethurst
add test follows to bun db test suite
abd2154
@tsmethurst
update notifications
4d1c7c8
@tsmethurst
updatebyprimarykey => updatebyid
5fe209b
@tsmethurst
fix session
a21f07d
@tsmethurst
prefer explicit ID to pk
21142f4
@tsmethurst
fix little fucky wucky
aabb58f
@tsmethurst
remove workaround
b77de7e
@tsmethurst
use proper db func for attachment selection
632e8d2
@tsmethurst
update status db
af11e5d
@tsmethurst
add m2m entries in test rig
053b34f
@tsmethurst
fix up timeline
b46eeff
@tsmethurst
go fmt
49514b7
@tsmethurst
fix status put issue
56ef830
@tsmethurst tsmethurst merged commit aa07750 into main Oct 8, 2022
@tsmethurst tsmethurst deleted the bun_ident_fixes branch October 8, 2022 11:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@NyaaaWhatsUpDoc NyaaaWhatsUpDoc Awaiting requested review from NyaaaWhatsUpDoc

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@tsmethurst