[BugFix] Use __alias_dict__ instead of Field(alias) for data fields.

[deeleeramone]

1. Why?:

   • OpenAPI gets confused by having Field(alias='someAlias') and will end up labeling the Field as the Alias unless we use__alias_dict__.

2. What?:

   • Moves the alias definition from Field to __alias_dict__

3. Impact:

   • A few Finviz field names have been adjusted to be more like the others.
   • Improved data model schemas in openapi.json

4. Testing Done:

   • Ran the test suites

[piiq]

@andrewkenreich @tehcoderer please take a look at this one to see if there are any changes introduced here are breaking for you.
Apart from the Finviz (which IIUC is not used by the app side) we don't expect the API to change, but it's better to be on the safe side and that's why I'm tagging you.

[piiq]

[andrewkenreich]

We are actually doing this already on our end, so this is good