`tuple delete` should support CSV files
Given tuple write supports CSV file, we should add support for CSVs in tuple delete for consistency reasons.
We'd be able to run the command below if we want to delete the tuples we just added
fga tuple write --file tuples.csv
fga tuple delete --file tuples.csv
I can attempt this, it will be my first issue / enhancement ever. It may take me some time, hopefully that's alright.
@kindlyturnips sounds great, I'll assign it to you. Thanks a lot!
Hi @aaguiarz, I have an update of what I have accomplished today. It's mostly getting all the dependencies and the project up and running.
Accomplishments
- Downloaded and launched the openfga server with docker
- Created stores & authorization models for the stores
- Successfully written a json
./dist/fga tuple write --store-id=<> --file ./cmd/tuple/testdata/tuples.json - Successfully written a csv
./dist/fga tuple write --store-id=<> --file ./cmd/tuple/testdata/tuples_without_optional_fields.csv
Tomorrow I will start working on the fga tuple delete --file tuples.csv . Thanks for your patience and allowing me to participate.
Hello @aaguiarz I think I have it solved. I'm having difficulty with my git push, It appears that I don't have the authority to do so. Could you check this out for me?
Hi @kindlyturnips
You need to follow the process described here:
https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/fork-a-repo#propose-changes-to-someone-elses-project
- Fork
- Push to their fork
- Open a PR
Hello @aaguiarz Thank you for your patience, I've submitted the pull request. Let me know if I made a mistake somewhere in the process. Thanks again for letting me contribute.