authgear / authgear/authgear-server
Review whether commit hooks should be fired for ReadOnly
- 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
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