[GitHub] [trafficcontrol] moltzaum edited a comment on issue #3580: TO: Added ReadBack function to fix "joined fields" issue

[GitBox <gi...@apache.org>]

moltzaum edited a comment on issue #3580: TO: Added ReadBack function to fix "joined fields" issue
URL: https://github.com/apache/trafficcontrol/pull/3580#issuecomment-493139728
 
 
   The servers endpoint isn't fixed because the server does not implement the `GenericReader`. Previously I had a case working that used `Read` instead of the `GenericReader`, but I had to change the function signature for Create to be: `Create() ([]interface{}, error, error, int)` like the Read function. I didn't like that approach since I don't think it makes sense intuitively for a Create method to return an array of interfaces, especially since it would not benefit endpoints that don't use the generic create method.
   
   Also, I'm not sure if there is a good way to turn an empty interface{} into an object of a certain type without reflection.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services