search

inboundnow / retired-leads

Track visitor activity, capture and manage incoming leads, and send collected emails to your email service provider for WordPress
http://www.inboundnow.com/leads/
11 stars 3 forks source link

Lead List CPT and Taxomonies #12

Closed DavidWells closed 10 years ago

DavidWells commented 11 years ago

I think we need to rethink the complexity of the CPT and Taxonomy syncing.

For example, if you change the list name after its created and leads are already on the list what happens with the leads meta key "wpleads_list_ids"

It contains {"list-1":{"list_id":"29","wplead_list_category_id":"2"}}

So this was "List 1" and then I changed the list to a new name. The slug no longer relates to the list and I foresee this breaking things in the future.

Do we really need a custom post type for viewing lists?

I think we could simply have an admin page and show the WP_table_class on it. (like what already shows on each CPT.)

It would be a single page with a get param with the taxonomy id. like http://local.dev/wp-admin/edit.php?post_type=wp-lead&page=wpleads_list_view&list_taxomony_id=12

atwellpub commented 11 years ago

There's a lot to consider. I am not sure what parts are dependent on the cpt id right now.

If we got rid of the cpt the section where we manage categories would have to appear or something that replaces it: http://localhost/inboundsoon/wp-admin/edit-tags.php?taxonomy=wplead_list_category

Also with CPT's we have the ability to have dashboard like stats inside the cpt view area.

For now I can make sure that the category data changes when the cpt list name is changed.

DavidWells commented 10 years ago

List custom post type is sunsetting. Taxonomy will control lists.