search

AY2425S1-CS2103-F13-2 / tp

MIT License
2 stars 5 forks source link

Clashing Prefix for Description and Date #76

Closed C5hives closed 2 hours ago

C5hives commented 2 hours ago

As mentioned in PR #70, there is currently a clash in the naming of prefixes for a description and a date. Both currently use the prefix d/.

The Vendor class is unlikely to contain a Date object, but an Event could possibly contain a Description object in the future.

What would be the best prefixes to use to distinguish these two fields? I would suggest:

This would be the least ambiguous in my opinion.

C5hives commented 2 hours ago

Date will use the prefix on/. The naming can be contested in the future if need be.