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

Table storage filtering #9

Merged
merged 4 commits into from
Apr 20, 2018
Merged

Table storage filtering #9

merged 4 commits into from
Apr 20, 2018

Conversation

Carl-Hugo
Copy link
Member

Added ForEvolve.Azure.Storage.Table.IFilterableTableStorageReader that allows reading from Azure Table Storage with custom filters.

@Carl-Hugo
Copy link
Member Author

This is incomplete.

Minimum features to implement:

  • TableQuery helpers like StartWith; Maybe something chainable like that: query.Column("ColName").StartWith("String to check").
  • See if it is possible to make a non generic IFilterableTableStorageReader that support ReadAsync<TModel>(...) instead of the class level generic model.
    • Possible problem: querying on multiple tables. The ITableStorageSettings is usually injected in the class constructor; See how to solve this.
  • Add ITableStorageRepositoryFactory to help lower the number of dependencies injected in classes:
    • ITableStorageRepository<TModel> CreateRepository<TModel>()
    • IFilterableTableStorageReader<TModel> CreateReader<TModel>()

@Carl-Hugo Carl-Hugo closed this Apr 20, 2018
@Carl-Hugo
Copy link
Member Author

Carl-Hugo commented Apr 20, 2018

Reopened to include in the dev builds

@Carl-Hugo Carl-Hugo reopened this Apr 20, 2018
@Carl-Hugo Carl-Hugo changed the base branch from master to dev April 20, 2018 16:35
@Carl-Hugo Carl-Hugo added this to In progress in ForEvolve Framework Apr 20, 2018
@Carl-Hugo Carl-Hugo merged commit ce00eaa into dev Apr 20, 2018
ForEvolve Framework automation moved this from In progress to Done Apr 20, 2018
@Carl-Hugo Carl-Hugo added this to the v1.0.0 milestone Apr 25, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

None yet

1 participant