OpenEugene / little-help-book-api

the little help book api and sample app
MIT License
1 stars 0 forks source link

Added support for City table and most needed Place fields #7

Closed colindavey closed 3 years ago

colindavey commented 3 years ago

Added support for City table, and most missing Place fields. Tested running the server locally.

This is a draft pull request so I can get feedback.

Ideally I would like to add support for the CatSubcats table and the two still-missing Place fields (City and CatSubcats) before merging. They are links to records in other tables. I wonder if the way subcategories are handled in the HelpModels.cs Category class is a model for how to handle this, namely these lines:

        [JsonProperty("Subcategories")]
        public List<string> Subcategories { get; set; } = new List<string>();
        public List<Subcategory> SubcategoryList { get; set; } = new List<Subcategory>();
markdav-is commented 3 years ago

great work!