manolo / james-hupa-vaadin

A Vaadin webmail using Hupa as backend.
8 stars 3 forks source link

Stacktrace on screen by not filling in IMAP and SMTP port #1

Open Peppe opened 10 years ago

Peppe commented 10 years ago

Steps to reproduce: 1) Go to demo 2) Hit Login

If you don't fill in the IMAP and SMTP port (hidden away with the cogwheel), you'll get a stack trace showing a NumberFormatException on the login button. Do validation of the settings before using them.

As a side not, most want to just click around on the demo without filling in actual adresses, like I did. See if you could fake the connection or provide a demo account which doesn't send out emails.

manolo commented 10 years ago

Original Hupa is thought for real accounts, and the normal way is after filling your email it fills your imap/smtp settings automatically. That works pretty well normally.

Anyway I agree, we need some demo account for facilitating people just test the application, I'll setup something.

Validation and messages should be improved so as users understand what is happening.

soundar-gnts commented 10 years ago

Hi manolo, Mail attachment not working.. is Vaadin upload component support?

manolo commented 10 years ago

I have not implemented it yet because I have to change somethings in hupa servlet, but vaadin supports it.