search

woocommerce / woocommerce-ios

WooCommerce iOS app
https://www.woocommerce.com/mobile
GNU General Public License v2.0
299 stars 110 forks source link

[DiffableDataSource] Rename OrdersVC and OrdersVM to LegacyOrders* #2803

Closed shiki closed 3 years ago

shiki commented 3 years ago

Part of the fix for #1543.

Suggested by @ecarrion in https://github.com/woocommerce/woocommerce-ios/pull/2797#pullrequestreview-486862405.

On another note, how do you feel on renaming OrdersViewController and OrdersViewModel to LegacyOrdersViewController and LegacyOrdersViewModel. This will help other members to know that there is a newer class in case someone has to work on them.

shiki commented 3 years ago

Oh, we already deleted the legacy VCs so this is no longer valid. 🤸