Settings shows Saved. while another page's RPC URL stays stored #688

Open
opened 2026-10-27 07:27:27 +01:00 by clawbot · 0 comments
Collaborator

Found by the third review of #684, outside its scope.

Where

Settings, the RPC "Save" in src/popup/views/settings.js.

What happens

Another popup page saves a different RPC URL for the same network. Then the user, on this page, presses "Save" on the URL this page loaded. Settings shows "Saved." but the other page's URL stays stored. Read from the code on next; not run in a browser.

What should happen

"Saved." is shown only when the URL on screen is the one stored. The save either stores the URL on screen or says it did not.

Severity: minor

Definition of done

  • After another page stores a different URL for the same network, "Save" on this page stores the URL on screen and shows "Saved.", or refuses with a plain sentence.
  • A test, failing on next.

Model: opus-5-5

Found by the third review of https://git.eeqj.de/sneak/AutistMask/pulls/684, outside its scope. ## Where Settings, the RPC "Save" in `src/popup/views/settings.js`. ## What happens Another popup page saves a different RPC URL for the same network. Then the user, on this page, presses "Save" on the URL this page loaded. Settings shows "Saved." but the other page's URL stays stored. Read from the code on `next`; not run in a browser. ## What should happen "Saved." is shown only when the URL on screen is the one stored. The save either stores the URL on screen or says it did not. Severity: minor ## Definition of done - [ ] After another page stores a different URL for the same network, "Save" on this page stores the URL on screen and shows "Saved.", or refuses with a plain sentence. - [ ] A test, failing on `next`. Model: opus-5-5
clawbot self-assigned this 2026-10-27 07:27:27 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sneak/AutistMask#688