Email Notifications not displaying
CPunch opened this issue · 2 comments
From a quick glance at Email.cpp, it looks like P_CL2FE_REQ_PC_EMAIL_UPDATE_CHECK is implemented, however the P_FE2CL_REP_PC_NEW_EMAIL response packet seems to have no effect?
The new email packet does seem to work (it plays a sound and shows the email icon in the menu) but only if the count field is non-zero. I'll validate that the number of unread emails is being correctly fetched from the DB. If so, there's no further action to take here.
Confirmed this is working as intended; we send the correct number of unread emails in the response packet and the client plays a sound + shows the email icon immediately if the number is non-zero.
Maybe what was throwing us off here is that the client sends an initial request after a minute and then subsequent requests every 10 minutes. I have a feeling the original server sent the P_FE2CL_REP_PC_NEW_EMAIL packet on email send if the recipient is online. Might be worth adding.