NOTICE: We've Moved to a Ticket System for Support

As of August 31, 2017 (12am EST) our support forums will be retired (read-only), and we will be moving to a support ticket system. This will allow us to better organize and answer support requests, and provide a more personalized experience as we assist our customers.

For the time being, we will leave our forums open for reading and learning while we work on creating a more robust Knowledge Base for everyone to use.

If you are a WC Vendors Pro customer please open a support ticket here. 

If you are a WC Vendors user please open a support ticket on the Wordpress.org forums.

The information on this forum is outdated and in most instances no longer relevant. Please be sure to check our documentation for the most up to date information.

https://docs.wcvendors.com/

Thank you to all of our customers!

This topic has 4 replies, 3 voices, and was last updated 7 years, 2 months ago by Jason.

Viewing 5 posts - 1 through 5 (of 5 total)

Author

Posts
December 6, 2016 at 7:08 am #49488
Jason
Participant
Hi,

I’m currently fiddling with moving the registration page within our theme.

Since Woocommerce uses a short-code for “my account” and cannot be moved, I’m editing the myaccount/form-login.php template to accommodate this.

I wish to use /my-account/?action=register as it’s the most logical change to make BUT… I spotted something weird.

Before the “Apply to become a vendor” checkbox would appear.. but then on my new page, it didnt.

I believe its legacy code from the old plugin but I tracked it down to wc-vendors/classes/front/signup/class-vendor-signup.php

Lines 42-46 inside vendor_option function:
```
		if ( isset( $_GET[ 'action'] ) && 'register' == $_GET[ 'action'] ) { 

			// include_once('views/html-vendor-signup.php'); 

		} else {
```
This class is checking for an the action and keyword Register…

It used to include a template but that is commented out – which removes it from my new register page.

Any chance we can remove this conditional or include the template again?

Thanks
January 20, 2017 at 7:42 am #53696

Jason
Participant

Hi guys,

Do you plan to make any changes to this? It would really be helpful to have an editable template and to allow use of the “action=register”, as part of themes.

January 20, 2017 at 3:18 pm #53736

Anna
Member

Hello,
I will submit this as an enhancement and see if this could be done so there is more flexibility with the vendor registration.

January 20, 2017 at 3:48 pm #53745

WC Vendors Support
Participant

action=register is probably just asking for conflicts with other plugins.

Perhaps what we could do is add an action in that space so you can execute arbitrary code through the action?

February 3, 2017 at 3:48 pm #55073

Jason
Participant

Hi Ben,

I’m actually asking for those lines to be removed from your plugin because they break that action hook.

Theme authors can use any combo they want.
Author

Posts