Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[new feature] Connection with Identity - Issue #73 #74

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

Conversation

kumkee
Copy link

@kumkee kumkee commented Dec 5, 2023

This is to implement the new feature suggested in Issue #73 - connection via default identity instead of keys.

Disclaimer

The addition has not been tested as I am still learning how to locally build and test a Nuget package. Any help will be appreciated. So far, I've only read about Nuget local feeds but have no clues on how to compile the whole repo into a .nupkg file.

It is supposed to work as follows

let findUsers() =
    host
    |> Cosmos.host
    |> Cosmos.connectWithIdentity
    |> Cosmos.database "UserDb"
    |> Cosmos.container "UserContainer"
    |> Cosmos.query "SELECT u.FirstName, u.LastName FROM u WHERE u.LastName = @name"
    |> Cosmos.parameters [ "@name", box "Powell" ]
    |> Cosmos.execAsync<User>

@kumkee kumkee changed the title [new feature] Connection with Identity - Issue https://github.com/aaronpowell/FSharp.CosmosDb/issues/73 [new feature] Connection with Identity - Issue #73 Dec 5, 2023
@bartelink
Copy link
Contributor

Perhaps the info in https://github.com/jet/dotnet-templates#testing and/or the build.proj in there might help with techniques and/or automation for testing the package

@aaronpowell
Copy link
Owner

Nah, something stuffed up in the paket.lock file that I'll have a look at when I have some time.

@kumkee
Copy link
Author

kumkee commented Feb 7, 2024

Nah, something stuffed up in the paket.lock file that I'll have a look at when I have some time.

Any progress?

@kumkee
Copy link
Author

kumkee commented Mar 4, 2024

The update has been tested.
If you can't wait for the merge and want to use connection with identity right now, you can use my NuGet packet which contains the update.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants