Skip to content
This repository has been archived by the owner on Jun 18, 2024. It is now read-only.

feat: folder endpoints #23

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

TristanBlackwell
Copy link

@TristanBlackwell TristanBlackwell commented Nov 26, 2023

Folder API endpoints

Implements the basic folder logic from the API. This will allow for the creation, listing, updating, and deletion of folders via the SDK.

Not for this PR but eventually this can be integrated more naturally into the SDK behaviour. For example, if a user creates a secret at a folder that does not yet exist then this folder may be created (possibly configurable).

This is my first PR for Infisicial so I encourage a thorough review of the alterations although I have atttempted to follow the standards & practices of the existing code.

Points to highlight:

  • I have made the decision to throw potential errors during folder operations. I believe there is some wider scope work on error handling to be done based on No errors #22 but there wasn't much to reference here since secrets have the fallback mechanism.

@TristanBlackwell TristanBlackwell marked this pull request as ready for review November 27, 2023 17:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant