luminus-framework / luminus-framework/conman

New option to wrap database hits

Open
#19 1 comment 2 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement
Dominant language
Clojure
Stars
127
Forks
25
PR merge metrics
No merged PRs in 30d

Description

It would be _really_ nice if a function could be provided to `conman/bind-connection` that would wrap database-touching functions.

Something like

``` clojure
(conman/bind-connection *db*
{:wrapper-func log-it}
"sql/queries.sql")
```

where a theoretical `create-user!` call would be defined as

``` clojure
(defn create-user! [map]
(REAL-create-user (log-it map)))
```

or something to that effect.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start at conman/bind-connection and inspect how SQL from sql/queries.sql becomes database-touching functions such as create-user!. Clarify the intended wrapper-func behavior using the log-it example, including what arguments it receives and which generated functions it affects. Done means the option has an agreed scope and wrapping behavior that is covered by tests.

Written by the indexing model from the issue text.

Assessment

Tech stack
clojure
Domain
backend, database
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.