patrickarmengol / somethingcoffee Goto Github PK

• remove map view from api docs with an argument for get decorator; check docs

• dwithin
• bbox
• knn

• PUT - all
• PATCH - partial

• operation_id arg
• name arg
• summary arg
• description arg

Adding roasters domain is a bit of a long term goal. Likely will be implemented as a many-to-many relationship with shops. IDK if I should add to map. Many coffeeshops are roasters already and making a distinction would be confusing.

My idea for the roaster field in the shop details page looks like one of the following:

# null case
roaster: unknown

# they roast their own beans
roaster: self

# roaster partner
roaster: <a href="/roaster/{roaster.id}">{roaster.name}</a>

I also have to think about the case where the shop has multiple roaster partners. Should shop's roaster be roasters?
Also, what if the shop roasts their own, but the roaster name differs from the shop name?

• multi-line hours of operation attribute to shop model & schema
• alembic migration

Currently, bogus example shops are being used to test. Curate shops and populate the db. It should be limited to one city to start out.

• add a simple frontend for the list supplied by /api/shops endpoint

• use cache arg in operation decorators where needed (like lists)

• info
• image
• small leaflet location

• add a homepage at /
• link to map and list views

use docker-compose to set up project for deployment

• web
• db
• traefik

• city
• country
• gmaps link
• osm link

Working with jinja isn't the most pleasant experience. I should look at moving the frontend to a framework like vue instead and just let litestar only handle the api. This would simplify routing as well.

• new table / model / schema
• alembic migration

include the following amenities tags:

• beans

• espresso-based

• batch brew

• hand drip

• cold brew

• snacks

• meals

• milk alternatives

• pet friendly

• free wifi

add a link to the shop details page in the marker popup

• can be "self" or name of partner roaster or none
• alembic migration