Gravity

Gravity copied to clipboard

Added Model Generation Tool.

hhristov-tsd

RESTDao Provider

5

mrobustelli

ImportAPI Dao/Provider

2

ImportAPI would be another nice target for BaseDTO mapping. On a simple level, the code could convert a set of DTOs into a DataTable ready to import via the ImportAPI....

Arithmomaniac

Document or remove string truncation

2

- Are we sure we want to truncate strings instead of throwing an exception? - If so, we should probably have documentation. - Why have any cutoff for a max-length...

Arithmomaniac

Need to update version of TestHelpers used.

1

There are some changes to the DBContext in the APIs that may cause some problems with tests run against later 9.6 versions. The update should not require Gravity coding changes.

mrobustelli

Split DAOs into own NuGet packages

1

Moving each provider into its own Nuget package has a few advantages: - Easier to navigate - Enforces separation of concerns - cross-DAO stuff cannot reference any Relativity-specific DLLs -...

Arithmomaniac

Need to clean up the documentation a bit.

3

I think there are some things we can do to make the documentation around the project a bit better. This issue should raise discussion around that.

mrobustelli

Use Polly for transient fault handling

4

Gravity currently has its own `InvokeWithRetryService`. It may be worth considering using [Polly](https://github.com/App-vNext/Polly) policies instead for this purpose, as it is battle-tested and more flexible.

Arithmomaniac

Add .ruleset file for Code Analysis

1

We now have coding standards, but not everything can be configured in editorconfig. Some things still have to be configured in [rulesets](https://docs.microsoft.com/en-us/visualstudio/code-quality/using-rule-sets-to-group-code-analysis-rules?view=vs-2017). We should create a rulesets file and add...

Arithmomaniac

Rename recursion level Enum and values

1

@brycekbargar proposed the following instead of the current names/values for `ObjectFieldsDepthLevel`: ```csharp ObjectFieldsBehavior.NoRecursion ObjectFieldsBehavior.ShallowRecursion ObjectFieldsBehavior.FullRecursion ``` I like it. After Fest, we can add these, and flag the `ObjectFieldsDepthLevel` as...

Arithmomaniac