Skip to content

release(apps/frontent/line-art-coloring): v0.1.0#95

Merged
99mini merged 3 commits intomainfrom
line-art-coloring/release/v0.1.0
Jun 2, 2025
Merged

release(apps/frontent/line-art-coloring): v0.1.0#95
99mini merged 3 commits intomainfrom
line-art-coloring/release/v0.1.0

Conversation

@99mini
Copy link
Owner

@99mini 99mini commented Jun 2, 2025

No description provided.

@99mini 99mini self-assigned this Jun 2, 2025
Copilot AI review requested due to automatic review settings June 2, 2025 04:35
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR prepares the line-art-coloring frontend for its v0.1.0 release by improving error handling around upload history retrieval.

  • Ensures history resets to an empty array on JSON parse errors instead of silently failing.
  • Filters out records older than one hour and enforces a maximum of five entries.

try {
history = JSON.parse(localStorage.getItem(KEY) || '[]');
} catch {}
} catch {
Copy link

Copilot AI Jun 2, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[nitpick] Consider capturing the error object in the catch block and logging it for easier debugging, e.g.: } catch (e) { console.warn('Upload history parse failed:', e); history = []; }

Suggested change
} catch {
} catch (e) {
console.warn('Upload history parse failed:', e);

Copilot uses AI. Check for mistakes.
@99mini 99mini merged commit 1eee9f1 into main Jun 2, 2025
2 checks passed
@99mini 99mini deleted the line-art-coloring/release/v0.1.0 branch June 2, 2025 04:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants