jeffio-coder
commented
1 year ago I had the same issue. I get the GET method to work. I still haven't been able to make the POST CreatePointOfInterest method work.
[HttpGet("{pointOfInterestId}", Name = "GetPointOfInterest")]
public ActionResult<PointOfInterestDto> GetPointOfInterest(int cityId, int pointOfInterestId)
{
var city = CitiesDataStore.Current.Cities.FirstOrDefault(c => c.Id == cityId);
if (city == null)
{
return NotFound();
}
var pointOfInterest = city.PointsOfInterest.FirstOrDefault(c => c.Id == pointOfInterestId);
if (pointOfInterest == null)
{
return NotFound();
}
return Ok(pointOfInterest);
}[HttpPost]
public ActionResult
var maxPointOfInterest = CitiesDataStore.Current.Cities.SelectMany(c => c.PointsOfInterest).Max(p => p.Id);
var finalPointOfInterest = new PointOfInterestDto()
{
Id = ++maxPointOfInterest,
Name = pointOfInterest.Name,
Description = pointOfInterest.Description
};
city.PointsOfInterest.Add(finalPointOfInterest);
return CreatedAtRoute("GetPointOfInterest", new { cityId = cityId, pointOfInterestId = finalPointOfInterest.Id });
}
hasona69
Hi Kevin, I am following along with your course just now and noticed that if you use lowercase in the template
[HttpGet("{pointofinterestid}", Name = "GetPointOfInterest")]Then it does not work in swagger, works fine in postman though. If you change the casing to match the variable[HttpGet("{pointOfInterestId}", Name = "GetPointOfInterest")]Then it works fine in both. Loving the course so far.