Flagsmith / Flagsmith/flagsmith

Actionable error message for duplicate priorities in update-flag

Open Beginner friendly
#8,301 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
6.6k
Forks
567
Avg merge
1d 14h
Merged PRs (30d)
124

Description

> One small nit left IMO: I'd make the error message actionable so users understand why we can't accept ambiguous priorities. Something among the lines of `The overrides for seg1 and seg2 are in conflict; provide explicit priority values.`

_Originally posted by @khvn26 in https://github.com/Flagsmith/flagsmith/pull/8102#discussion_r3787215841_

Contributor guide

Open the contributing guide

Research direction

Locate the update-flag handling for duplicate priorities and inspect the current error path. Change the message so it explains the conflict and tells users to provide explicit priority values, then confirm that duplicate priorities still produce an actionable error.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
backend
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
65/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.