C22 Sphinx Priyanka

[PriyankaKaramchandani]

No description provided.

[yangashley]

Nice work! Waves 1 and 2 look good and the project directory is correctly organized.

You have some inconsistent white spacing and missed some indentations. Be mindful of this so that your code is polished and follows the PEP8 styleguide.

[yangashley]

How would you write the to_dict instance method so you can take an instance of Planet and turn it into a dictionary that will be returned as a response to a client's request?

[yangashley]

Suggested change

	planets =[
	planets = [

Nit: missing whitespace after =

[yangashley]

Inconsistent white spaces

Suggested change

	result_planets.append(
	{
	"id" : planet.id,
	"name": planet.name,
	"description" : planet.description,
	"distance_from_sun" : planet.distance_from_sun
	}
	result_planets.append(
	{
	"id": planet.id,
	"name": planet.name,
	"description": planet.description,
	"distance_from_sun": planet.distance_from_sun
	}

[yangashley]

Needs one more level of indentation

Suggested change

	return {
	"id" : planet.id,
	"name" : planet.name,
	"description" : planet.description,
	"distance_from_sun" : planet.distance_from_sun
	}
	return {
	"id" : planet.id,
	"name" : planet.name,
	"description" : planet.description,
	"distance_from_sun" : planet.distance_from_sun
	}