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

"No_Email" Field Contact : visibility -1 to 0 #26518

Closed

Conversation

atm-lena
Copy link
Contributor

@atm-lena atm-lena commented Nov 9, 2023

FIX|Fix

The field must not be visible : "-1" value bug => field is visible on list.

@eldy
Copy link
Member

eldy commented Nov 9, 2023

Why this field must not be visible ? It is a property we see on the file of a contact:
image
So why not on list ?

@eldy eldy added the Discussion Some questions or discussions are opened and wait answers of author or other people to be processed label Nov 9, 2023
@atm-lena
Copy link
Contributor Author

atm-lena commented Nov 9, 2023

Why this field must not be visible ? It is a property we see on the file of a contact: image So why not on list ?

If "no_email" is not visible, field in list does not disappear :
image

If "no_email" visible, there are 2 same column proposed on list :
image

One works, the other doesn't.
The column that works :
image

The column that doesn't works is "no_email" field.

These fields are not very clear... but they create a duplicate in the list

eldy added a commit that referenced this pull request Nov 14, 2023
@eldy eldy closed this Nov 14, 2023
@eldy
Copy link
Member

eldy commented Nov 14, 2023

I see. I suggested a better fix by completely removing this field as it is no more used now. Because unsusbcribe is stored into the dedicated table llx_mailing_unsubscribe.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Discussion Some questions or discussions are opened and wait answers of author or other people to be processed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants