search

serokell / coffer

Multi-backend password store with multiple frontends
4 stars 2 forks source link

Rename effects in the BackendEffect from `Secret` to`Entry` #20

Closed MagicRB closed 2 years ago

MagicRB commented 2 years ago

readSecret and so on don't precisely reflect the terminology we have agreed on. Changing them now would be a rather large change which would effect the whole codebase. Therefore we should wait for the `"big merge" until we tackle this.

sancho20021 commented 2 years ago

Do we need to rename all occurrences of word secret to entry in our project?

sancho20021 commented 2 years ago

@dcastro or @MagicRB , I made a draft pull request, cause I'm unsure whether my changes properly fix this issue. Please, have a look.