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

Checkin/out mail notification error #15754

Open
2 tasks done
igorpavlic opened this issue Oct 30, 2024 · 5 comments
Open
2 tasks done

Checkin/out mail notification error #15754

igorpavlic opened this issue Oct 30, 2024 · 5 comments
Assignees

Comments

@igorpavlic
Copy link

Debug mode

Describe the bug

Hi,
When checkinin or ot assets there is an error in log and mail does not arrive.
Acceptance mail is ok, reminders are ok.
Error is:
[2024-10-30 13:15:34] production.WARNING: Exception caught during checkin notification: Undefined property: App\Events\CheckoutableCheckedIn::$checkedOutBy

Reproduction steps

  1. Clixk Checkin on asset

Expected behavior

Mail notification

Screenshots

No response

Snipe-IT Version

7.0.13

Operating System

Windows 2022

Web Server

IIS

PHP Version

8.3.12

Operating System

No response

Browser

No response

Version

No response

Device

No response

Operating System

No response

Browser

No response

Version

No response

Error messages

[2024-10-30 13:15:34] production.WARNING: Exception caught during checkin notification: Undefined property: App\Events\CheckoutableCheckedIn::$checkedOutBy

Additional context

No response

Copy link

welcome bot commented Oct 30, 2024

👋 Thanks for opening your first issue here! If you're reporting a 🐞 bug, please make sure you include steps to reproduce it. We get a lot of issues on this repo, so please be patient and we will get back to you as soon as we can.

@DarrenRainey
Copy link
Contributor

How was you system installed - is this a fresh install using the manual process, a previous install or have use used the installer script on a clean system?

@igorpavlic
Copy link
Author

igorpavlic commented Oct 31, 2024 via email

@snipe
Copy link
Owner

snipe commented Oct 31, 2024

That's just a warning, so that part is fine. Have you upgraded to latest on master? We merged some changes to the notification system yesterday.

@igorpavlic
Copy link
Author

igorpavlic commented Nov 4, 2024 via email

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

4 participants