search

foivz / pop-app-flutter

Mobile application for the course Entrepreneurial venture.
2 stars 0 forks source link

Dev/main menu/feature 5 implement buyer menu #18

Closed mislavmatijevic closed 1 year ago

mislavmatijevic commented 1 year ago

Introduces minor additions to the already existing main menu. Basically changes the name of the first option ("BUY") and opens a dialog on click of the option that is going to be implemented in #11.

This PR aims to connect seller menu (#4) and buyer menu (#5) features into a single ("seller") branch. From there the seller branch would (possibly ASAP) be merged into main and we could keep working on remaining functionalities from there.

Current solution leaves space for implementing a design pattern such as Factory Method to choose whether the main menu's first button is going to be a SELL or a BUY option. Mixin could also do the work. However, this remains to be implemented AFTER the two branches are merged with 'main' branch.

mislavmatijevic commented 1 year ago

I haven't integrated the grid changes owning to it being too complicated and unnecessary at this point.