← Back to issue list

errors: expose the StoreServerError cause from the returned json (CRAFT-582)

View original Github issue

Metadata

Project
craft-store
Number
#12
Type
pull request
State
merged
Author
sergiusens
Labels
—
Created
Updated
Closed

Current evaluation

Merged canonical/craft-store#12, exposing StoreServerError causes from returned JSON and adding expanded error handling and tests.

Resolution: closed resolved

Reason: PR #12 was approved and merged, implementing the StoreServerError cause exposure for CRAFT-582.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com> ------- - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? -----

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged canonical/craft-store#12, exposing StoreServerError causes from returned JSON and adding expanded error handling and tests.
stealth/space-bunny-alpha — closed resolved Merged canonical/craft-store#12 via commit 329b641, exposing StoreServerError causes from returned JSON with expanded error handling and tests.
stealth/space-bunny-alpha — closed resolved Merged in canonical/craft-store#12, exposing the StoreServerError cause in returned JSON to resolve CRAFT-582 (+135/-9 across three files).
qwen/qwen3.6-35b-a3b — — Exposed the StoreServerError cause from returned JSON. Merged after approval and passing CI checks. Modified three files (+135/-9 lines) to resolve CRAFT-582.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#61 errors: catch the correct JSONDecodeError craft-store merged Merged in craft-store#61 via commit e2ce4b5, correcting JSONDecodeError handling for CRAFT-1238 across three files.
74%
#670 errors: catch all CraftStoreErrors (CRAFT-813) charmcraft merged Merged PR #670 broadened craft_store initialization error handling to catch all CraftStoreErrors; implementation fe3215e3 landed via merge commit 677b399f.
73%