Core: Validate email before sending out public link from dialog

Created on 20 Jul 2017  路  4Comments  路  Source: owncloud/core

Steps

  1. Create a public Link
  2. Enter a wrong Mail Address (e.g. without @)
  3. Click save

Expected

Link not saved, error message, dialog still there.

Actual

Dialog reappears four times.

Version

10.0.1

@felixheidecke can you have a look ?

Bug p2-high

All 4 comments

The approach used to fix this issue doesn't tell the user what is the problem with the email.
It is not possible to turn red this text field when it is not valid?

screen shot 2017-11-28 at 15 26 43

Or change the text here and say the email introduced is not valid?

screen shot 2017-11-28 at 14 08 15

@PVince81 @felixheidecke

Seems the PR only fixes the issue about the four duplicated dialogs but doesn't fix the UX.

I see the ticket is already set to 10.0.3 as it was a 10.0.3 fix. @SergioBertolinSG please reopen a separate ticket and set to QA to have it fixed for 10.0.4. Thanks

New ticket for 10.0.4 here: https://github.com/owncloud/core/issues/29703

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

Was this page helpful?
0 / 5 - 0 ratings