alirizaadiyahsi / Nucleus

Vue startup application template that uses ASP.NET Core API layered architecture at the back-end and JWT based authentication
MIT License
355 stars 86 forks source link

Update BaseController.cs #168

Closed edcastanohunter closed 3 years ago

edcastanohunter commented 3 years ago

I have been exploring your code and it looks really great. I just want to propose this change due that you are using here an API controller, and it would be better to use the ControllerBase class rather than Controller, this is due that ControllerBase doesn't handle thing like response with views. Hope this helps.