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

[#805] Implemented the Echo Store. #807

Merged
merged 3 commits into from
Oct 4, 2024
Merged

Conversation

Thijs153
Copy link
Contributor

resolves #805 with an Echo Store that returns a new tenant instance based on the given identifier.

@Thijs153
Copy link
Contributor Author

@dotnet-policy-service agree

@AndrewTriesToCode
Copy link
Contributor

@Thijs153 Looks great! I'm sorry I didn't realize a different PR I merged would cause conflicts. Can you resolve and I will merge ASAP? The conflicts should be very minimal.

Thijs153 added 2 commits April 30, 2024 10:23
# Conflicts:
#	docs/Stores.md
#	test/Finbuckle.MultiTenant.Test/DependencyInjection/MultiTenantBuilderExtensionsShould.cs
@Thijs153
Copy link
Contributor Author

Hey @AndrewTriesToCode, a little late, but I just saw your message. I resolved the conflicts, and I believe it should be good now. Let me know 🙂

@AndrewTriesToCode
Copy link
Contributor

Thanks!

@AndrewTriesToCode AndrewTriesToCode merged commit a3e5eee into Finbuckle:main Oct 4, 2024
10 checks passed
github-actions bot pushed a commit that referenced this pull request Oct 12, 2024
# [8.0.0](v7.0.2...v8.0.0) (2024-10-12)

* feat!: version policy update ([#888](#888)) ([487a3a6](487a3a6))

### Bug Fixes

* BasePathStrategy no longer breaks the strategy chain ([#884](#884)) ([3263eff](3263eff))
* prevent duplicate key annotation in AdjustKey() ([#883](#883)) ([f75ba2c](f75ba2c))
* removed unused parameter from WithPerTenantRemoteAuthenticationConvention ([#886](#886)) ([dd17ab5](dd17ab5))

### Features

* add GetAllAsync() support for HttpRemoteStore ([#848](#848)) ([4208149](4208149))
* added the Echo Store. ([#807](#807)) ([a3e5eee](a3e5eee))
* strategies return null on invalid context type ([#885](#885)) ([9834575](9834575))

### BREAKING CHANGES

* This commit brings the release into alignment with the new version policy. See #887 for details.
* Included strategies for ASP.NET Core would throw an exception if the passed context was not an `HttpContext` type. Now they will return null indicating no identifier was found.
@AndrewTriesToCode
Copy link
Contributor

🎉 This PR is included in version 8.0.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

[Question] MultiTenancy without stores?
2 participants