Extract local storage management to hook

ScottLogic / prompt-injection

Application which investigates defensive measures against prompt injection attacks on an LLM, with a focus on the exposure of external tools.

MIT License

16 stars 10 forks source link

Closed chriswilty closed 7 months ago

chriswilty commented 7 months ago

App.tsx is really large, as it handles

I've extracted the local storage handling into a hook.

Have you done the following?

[x] Linked the relevant Issue (done originally as part of prep for authentication in UI: #807)
[x] Added tests
[x] Ensured the workflow steps are passing

chriswilty commented 7 months ago

@pmarsh-scottlogic Yeah, next step would be to extract all that dialog stuff, though right now I'm struggling to see a nice way of achieving that...

chriswilty commented 7 months ago

Just realised I've been lazy and not added any unit tests for the hook... I'll do that shortly.

pmarsh-scottlogic commented 7 months ago

exemplary work on the tests