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

Add a filter to allow accessing the Print link in Emails to Non-logged in users. #418

Open
priyanka-tychesoftwares opened this issue Dec 16, 2024 · 1 comment · May be fixed by #419
Open

Comments

@priyanka-tychesoftwares
Copy link
Contributor

priyanka-tychesoftwares commented Dec 16, 2024

Add a filter to allow accessing the Print link in Emails to Non-logged users.

One client wants to allow his users to view and print the receipt immediately upon clicking the print link in the email, without requiring a login. He mentioned that "This behavior was convenient for users and posed no issues in my experience.
Requiring a login to view and print the receipt introduces unnecessary inconvenience for users. My clients do not wish for such additional steps either. Since the email containing the link is sent directly to the user, it is reasonable to assume that only the intended recipient will access it. Anticipating scenarios where someone else might see the email and link seems like a separate concern and should not impose restrictions on legitimate users."

Additional context
https://support.tychesoftwares.com/conversation/7282?folder_id=15

@gaminikhil gaminikhil modified the milestones: Future Milestone, 5.5.0 Dec 17, 2024
gaminikhil added a commit that referenced this issue Dec 17, 2024
Fix #418 added 'allow_user_email_order_access' filter for non login user
@gaminikhil gaminikhil linked a pull request Dec 17, 2024 that will close this issue
@upenkapse upenkapse modified the milestones: 5.4.1, Future Milestone Dec 18, 2024
@gingerlime
Copy link

+1 my shops are selling without requiring an account at all, so if the link requires login my customers have no way to access the invoice at all.

@upenkapse upenkapse modified the milestones: Future Milestone, 5.5.0 Jan 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants