Add EvidenceStore for tracking activity evidence (#496)#501
Add EvidenceStore for tracking activity evidence (#496)#501sawankshrma wants to merge 5 commits intodevsecopsmaturitymodel:mainfrom
Conversation
vbakke
left a comment
There was a problem hiding this comment.
At first glance this looks good. But how do you execute the code? I didn't see how to initialize anything? Do you read the yaml file, or from localStorage?
6f09967 to
4bf1a80
Compare
Earlier, I was trying to mimic the structure of the
I don't know if this is the approach/merge-strategy we should go for. So I would genuinly like your opinion here @wurstbrot , @vbakke.
2026-02-26-08-45-49_BglElByh.mp4 |
|
😊 I struggled with that, too, @sawankshrma :) Made an obese solution, with timestamp for what was stored when etc. Ended up scrapping it. Kept it simple. I'd say normal workflow would be:
In other words, I think that:
If we want to go overboard with new functionality, we can somehow display what is newer in the localStorage vs server side yaml. Do say what should be exported, and saved to the server. But I think that is a separate issue, which is better solved after the evidence issue has been implemented. |
4bf1a80 to
a1e07b1
Compare
Here's a quick summary of what this PR does end to end:
I believe this is ready to merge. Once merged, I'll start on the UI layer implementation. Plz let me know :) |
vbakke
left a comment
There was a problem hiding this comment.
Looks good. Just some smaller comments. Feel free to continue with the UI :)
eb162f6 to
82e7bc1
Compare
Co-authored-by: vbakke <vbakke@users.noreply.github.com>
Addressed the comments in the last two commits. Will be opening a UI PR shortly. |
Description
Adds an
EvidenceStore.tsto let users record that activities have been completed. Evidence is stored separately from progress in localStorage.Format:
What's New?
evidence-store.ts— new store with add, update, delete (by id), merge, rename team, import/export functions.team-evidence.yaml— empty placeholder for YAML-based evidence.EvidenceStoreinto the existingDataStore.progress-store.ts.Resolves (Partially) : #496