authgear / authgear/authgear-server

Review whether commit hooks should be fired for ReadOnly

Open
#2,931 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Go
Stars
2k
Forks
125
Avg merge
2d 17h
Merged PRs (30d)
32

Description

- List out the places where ReadOnly is used.
- List out the events that require WithTx. Then (all events) - (events that require WithTx) are the events that could work with ReadOnly
- Study if each event that could work with ReadOnly WILL NOT be broken if commit hooks ARE NOT fired.
- Do not fire commit hooks for ReadOnly

Contributor guide

Open the contributing guide

Research direction

Start by listing every place ReadOnly is used, then identify which events require WithTx. Compare the remaining events with their commit-hook behavior and verify that omitting hooks will not break them. Done means the supported ReadOnly behavior is established and commit hooks are not fired for ReadOnly.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
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.