New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Should EDD Have a Profile Editor? #503

Closed
pippinsplugins opened this Issue Nov 16, 2012 · 16 comments

Comments

Projects
None yet
6 participants
@pippinsplugins
Member

pippinsplugins commented Nov 16, 2012

I've been having a discussion with a few users on whether or not EDD should forms to change passwords and update basic profile fields (email, name, display name, etc).

What do you guys think? Yes? No?

@sunnyratilal

This comment has been minimized.

Show comment
Hide comment
@sunnyratilal

sunnyratilal Nov 17, 2012

Member

Definitely something that should be integrated.

On 16 Nov 2012, at 22:25, Pippin Williamson notifications@github.com wrote:

I've been having a discussion with a few users on whether or not EDD should forms to change passwords and update basic profile fields (email, name, display name, etc).

What do you guys think? Yes? No?


Reply to this email directly or view it on GitHub.

Member

sunnyratilal commented Nov 17, 2012

Definitely something that should be integrated.

On 16 Nov 2012, at 22:25, Pippin Williamson notifications@github.com wrote:

I've been having a discussion with a few users on whether or not EDD should forms to change passwords and update basic profile fields (email, name, display name, etc).

What do you guys think? Yes? No?


Reply to this email directly or view it on GitHub.

@samikeijonen

This comment has been minimized.

Show comment
Hide comment
@samikeijonen

samikeijonen Nov 17, 2012

Collaborator

Yes from me too.

Collaborator

samikeijonen commented Nov 17, 2012

Yes from me too.

@pippinsplugins

This comment has been minimized.

Show comment
Hide comment
@pippinsplugins

pippinsplugins Nov 19, 2012

Member

Should it include any fields besides those I mentioned above?

Member

pippinsplugins commented Nov 19, 2012

Should it include any fields besides those I mentioned above?

@sunnyratilal

This comment has been minimized.

Show comment
Hide comment
@sunnyratilal

sunnyratilal Nov 19, 2012

Member

That should be ample. You should then just add an action for others to then extend those fields should they need them.

Member

sunnyratilal commented Nov 19, 2012

That should be ample. You should then just add an action for others to then extend those fields should they need them.

@pippinsplugins

This comment has been minimized.

Show comment
Hide comment
@pippinsplugins

pippinsplugins Nov 19, 2012

Member

Setting this with a milestone of v1.3.4.

Member

pippinsplugins commented Nov 19, 2012

Setting this with a milestone of v1.3.4.

@sunnyratilal

This comment has been minimized.

Show comment
Hide comment
@sunnyratilal

sunnyratilal Nov 19, 2012

Member

Great! Looking forward to it. I'm happy to do parts of it if you want?

Sent from my iPhone

On 19 Nov 2012, at 13:54, Pippin Williamson notifications@github.com wrote:

Setting this with a milestone of v1.3.4.


Reply to this email directly or view it on GitHub.

Member

sunnyratilal commented Nov 19, 2012

Great! Looking forward to it. I'm happy to do parts of it if you want?

Sent from my iPhone

On 19 Nov 2012, at 13:54, Pippin Williamson notifications@github.com wrote:

Setting this with a milestone of v1.3.4.


Reply to this email directly or view it on GitHub.

@spencerfinnell

This comment has been minimized.

Show comment
Hide comment
@spencerfinnell

spencerfinnell Nov 19, 2012

Member

If this is implemented, then the fields should also be added to the user profile on the admin side as well, so site admins can manually edit/update people's information.

Member

spencerfinnell commented Nov 19, 2012

If this is implemented, then the fields should also be added to the user profile on the admin side as well, so site admins can manually edit/update people's information.

@pippinsplugins

This comment has been minimized.

Show comment
Hide comment
@pippinsplugins

pippinsplugins Nov 19, 2012

Member

I don't think there should be any fields that aren't already in the admin.

Member

pippinsplugins commented Nov 19, 2012

I don't think there should be any fields that aren't already in the admin.

@sunnyratilal

This comment has been minimized.

Show comment
Hide comment
@sunnyratilal

sunnyratilal Nov 19, 2012

Member

I agree with Pippin.

Sent from my iPhone

On 19 Nov 2012, at 14:36, Pippin Williamson notifications@github.com wrote:

I don't think there should be any fields that aren't already in the admin.


Reply to this email directly or view it on GitHub.

Member

sunnyratilal commented Nov 19, 2012

I agree with Pippin.

Sent from my iPhone

On 19 Nov 2012, at 14:36, Pippin Williamson notifications@github.com wrote:

I don't think there should be any fields that aren't already in the admin.


Reply to this email directly or view it on GitHub.

@chriscct7

This comment has been minimized.

Show comment
Hide comment
@chriscct7
Member

chriscct7 commented Dec 8, 2012

+1

@corsonr

This comment has been minimized.

Show comment
Hide comment
@corsonr

corsonr Dec 17, 2012

Contributor

Agree with Pippin, that's the aim of the plugin, so only core fields should be taken into account.

Contributor

corsonr commented Dec 17, 2012

Agree with Pippin, that's the aim of the plugin, so only core fields should be taken into account.

@chriscct7

This comment has been minimized.

Show comment
Hide comment
@chriscct7

chriscct7 Dec 17, 2012

Member

We've got this in Jigoshop. Shouldn't be hard to port it.

Member

chriscct7 commented Dec 17, 2012

We've got this in Jigoshop. Shouldn't be hard to port it.

@sunnyratilal

This comment has been minimized.

Show comment
Hide comment
@sunnyratilal

sunnyratilal Dec 23, 2012

Member

I'll start by this and this Profile Editor will essential work via the use of the shortcodes.

Member

sunnyratilal commented Dec 23, 2012

I'll start by this and this Profile Editor will essential work via the use of the shortcodes.

@sunnyratilal

This comment has been minimized.

Show comment
Hide comment
@sunnyratilal

This comment has been minimized.

Show comment
Hide comment
@sunnyratilal

sunnyratilal Dec 23, 2012

Member

Profile Editor Completed See here: #617

Member

sunnyratilal commented Dec 23, 2012

Profile Editor Completed See here: #617

@pippinsplugins

This comment has been minimized.

Show comment
Hide comment
@pippinsplugins

pippinsplugins Dec 23, 2012

Member

See my notes please.

Member

pippinsplugins commented Dec 23, 2012

See my notes please.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment