Open itsnotsherm opened 1 week ago
While it may appear that the f/Price parameter is currently redundant due to the existence of only one sorting field (Price), its inclusion serves an important foundational purpose for the application's scalability and maintainability.
The f/ parameter is deliberately designed to accommodate additional sorting fields that may be introduced in subsequent updates, such as sorting by selling price, buying price, property size, or house name. By implementing this parameter now, we maintain consistency and avoid disruptive changes to the command syntax in the future. Removing the field now and reintroducing it later would confuse users and create unnecessary friction.
Team chose [response.NotInScope
]
Reason for disagreement: [replace this with your explanation]
if there's only one field sorti uses to sort the properties, the field is redundant and inconveniences the user, can remove it entirely and have the user just state whether they want the properties sorted in ascending or descending order of prices
not optimized for fast-typists