Skip to content
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

User Image: New Photo Entity #13

Open
Tonterias opened this issue Sep 16, 2019 · 3 comments
Open

User Image: New Photo Entity #13

Tonterias opened this issue Sep 16, 2019 · 3 comments

Comments

@Tonterias
Copy link
Collaborator

No description provided.

@Tonterias
Copy link
Collaborator Author

A User should have another entity for his photo. Now it is mixed on the ChatUser, so we need to let the user access the ChatUser to change the photo, but that is a problem because they can mess with the relation to the JHIUser and cause a problem. Once separated the USER will not be able to Edit/Delete the ChatUser once it is created.

@Tonterias Tonterias added enhancement New feature or request new feature and removed enhancement New feature or request labels Sep 16, 2019
@cchunduri
Copy link
Collaborator

I don't think so we need to create another entity for store user profile image data. Because Jhipster User object itself contains a parameter called imgurl which can hold location to profile pic URL. All we need is a neat UI for uploading profile pic.

@Tonterias Tonterias added this to the Spingular Chat V1 Alpha milestone Sep 19, 2019
@Tonterias
Copy link
Collaborator Author

Tonterias commented Sep 24, 2019

Ok, how would you do it? Would it be easier to create such IU?

I think the new Entity Photo #18 is a good approach because Jhipster will do all the programming for us, but if you think that your approach is easier / cleaner / faster. It is ok with me.

Thanks,

@Tonterias Tonterias changed the title NEW Feature: User Image User Image Sep 24, 2019
@Tonterias Tonterias changed the title User Image User Image: New Photo Entity Sep 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants