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

"Allow Images" link missing/hidden #985

Closed
PaulTGG opened this issue Apr 29, 2024 · 10 comments
Closed

"Allow Images" link missing/hidden #985

PaulTGG opened this issue Apr 29, 2024 · 10 comments
Assignees

Comments

@PaulTGG
Copy link
Contributor

PaulTGG commented Apr 29, 2024

🐛 Bugreport

Version & Environment

Rev: 57756bc

OS: Debian 12.5

Steps to reproduce

  1. Install latest master.
  2. Look at an email that has images.
  3. There's no link to allow images.

Issue #873 was designed to allow images from a particular sender, but if the link is there it isn't obvious where.

@marclaporte
Copy link
Member

@jacob-js please advise :-)

@PaulTGG
Copy link
Contributor Author

PaulTGG commented May 1, 2024

@jacob-js More info - the links are only missing from the inline view. Both links (allow, and always allow) appear in the not-inline view.

@mercihabam
Copy link
Member

@jacob-js More info - the links are only missing from the inline view. Both links (allow, and always allow) appear in the not-inline view.

This could probably be the case. Let me give it a look.
Thanks @PaulTGG

@mercihabam
Copy link
Member

@PaulTGG please help me! How do you toggle the inline view?

@PaulTGG
Copy link
Contributor Author

PaulTGG commented May 3, 2024

@jacob-js I use these settings:
setting

...and I open the email from the inbox by clicking on the subject line:
inlineview

@mercihabam
Copy link
Member

@PaulTGG suprisingly I don't have such settings.
Screenshot Settings

I don't understand.
Is your version other than the latest master?

@PaulTGG
Copy link
Contributor Author

PaulTGG commented May 3, 2024

@jacob-js I'm running the latest master: 492f634. Also, I did not do an in-place upgrade. I use a database, and my database didn't change, but the front-end cypht folder was completely rebuilt from scratch.

@PaulTGG
Copy link
Contributor Author

PaulTGG commented May 3, 2024

@jacob-js Do you have the "inline_messages" module active in your .env file? That module is missing from the .env.example file, so you'd have to add it manually.

@mercihabam
Copy link
Member

@jacob-js Do you have the "inline_messages" module active in your .env file? That module is missing from the .env.example file, so you'd have to add it manually.

Thank you! This one fixed.
I re-confirm the reported issue, let's fix...

@mercihabam
Copy link
Member

Closing the issue, a PR has been created.
#1002

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants