Email field from statistics form should be supported by regexp

Bug #1387184 reported by Andrey Sledzinskiy
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Medium
Kate Pimenova

Bug Description

{

    "build_id": "2014-10-28_20-41-44",
    "ostf_sha": "f47fd1d66a7255213ee075d5c11b8f111f922000",
    "build_number": "278",
    "auth_required": true,
    "api": "1.0",
    "nailgun_sha": "31334a843d91d9b916d82dc2c746fd14bbb32882",
    "production": "docker",
    "fuelmain_sha": "e9839f68552ec0e8f10782a9a264d3a5b9f19ff5",
    "astute_sha": "97eea90efe0a1f17b4934919d6e459d270c10372",
    "feature_groups": [
        "mirantis"
    ],
    "release": "6.0",
    "release_versions": {
        "2014.2-6.0": {
            "VERSION": {
                "build_id": "2014-10-28_20-41-44",
                "ostf_sha": "f47fd1d66a7255213ee075d5c11b8f111f922000",
                "build_number": "278",
                "api": "1.0",
                "nailgun_sha": "31334a843d91d9b916d82dc2c746fd14bbb32882",
                "production": "docker",
                "fuelmain_sha": "e9839f68552ec0e8f10782a9a264d3a5b9f19ff5",
                "astute_sha": "97eea90efe0a1f17b4934919d6e459d270c10372",
                "feature_groups": [
                    "mirantis"
                ],
                "release": "6.0",
                "fuellib_sha": "efac0ed57d687f5d300cf277d477e57d2c646904"
            }
        }
    },
    "fuellib_sha": "efac0ed57d687f5d300cf277d477e57d2c646904"

}

Steps:
1. Install fuel
2. Open fuel UI
3. Log in
4. On Fuel UI leave fields empty and click 'Start Using Fuel'

Expected - empty fields are highlighted and message that field shouldn't be empty is near every empty field
Actual - there is only one message about empty field at the bottom (see attached screen )
Another issue to fix - email isn't validated. User can make a mistake by accident and he won't get assistance from support team

Tags: ui statistics
Revision history for this message
Andrey Sledzinskiy (asledzinskiy) wrote :
Changed in fuel:
status: New → Confirmed
Revision history for this message
Julia Aranovich (jkirnosova) wrote :

All empty text fields highlighted and the first error message displayed - this behaviour is by design. But can be discussed.

And +1 for adding email validation.

tags: added: ui
Changed in fuel:
milestone: 6.0 → 6.1
summary: - Incorrect validation of fields on enable/disable statistics page
+ Email field from statistics form should be supported by regexp
Changed in fuel:
assignee: Fuel UI Team (fuel-ui) → Kate Pimenova (kpimenova)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (master)

Fix proposed to branch: master
Review: https://review.openstack.org/147149

Changed in fuel:
status: Confirmed → In Progress
Changed in fuel:
assignee: Kate Pimenova (kpimenova) → Przemyslaw Kaminski (pkaminski)
Changed in fuel:
assignee: Przemyslaw Kaminski (pkaminski) → Kate Pimenova (kpimenova)
Changed in fuel:
assignee: Kate Pimenova (kpimenova) → Przemyslaw Kaminski (pkaminski)
Changed in fuel:
assignee: Przemyslaw Kaminski (pkaminski) → Kate Pimenova (kpimenova)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (master)

Reviewed: https://review.openstack.org/147149
Committed: https://git.openstack.org/cgit/stackforge/fuel-web/commit/?id=cb7312caadbf560a102742fac95eacdac6b5ad6b
Submitter: Jenkins
Branch: master

commit cb7312caadbf560a102742fac95eacdac6b5ad6b
Author: Kate Pimenova <email address hidden>
Date: Wed Jan 14 16:31:28 2015 +0300

    Fix for statistics agreement form validation

    - added migration for master_node_settings table

    Closes-Bug:#1387184

    Change-Id: Ie7932d3857892f8c5659a689b091fffb523d1f08

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
Anastasia Palkina (apalkina) wrote :

Verified on ISO #143

"build_id": "2015-02-24_22-54-44", "ostf_sha": "1a0b2c6618fac098473c2ed5a9af11d3a886a3bb", "build_number": "143", "release_versions": {"2014.2-6.1": {"VERSION": {"build_id": "2015-02-24_22-54-44", "ostf_sha": "1a0b2c6618fac098473c2ed5a9af11d3a886a3bb", "build_number": "143", "api": "1.0", "nailgun_sha": "f4487769f5526bd50ecd42af296f5587b7fc231f", "production": "docker", "python-fuelclient_sha": "5657dbf06fddb74adb61e9668eb579a1c57d8af8", "astute_sha": "6d6ad68e0cde286d74ac7d52e21da4fc8dcbe9ab", "feature_groups": ["mirantis"], "release": "6.1", "fuelmain_sha": "b975019fabdb429c1869047df18dd792d2163ecc", "fuellib_sha": "8384b8ca4db84794fb21e287202f05e31f78841c"}}}, "auth_required": true, "api": "1.0", "nailgun_sha": "f4487769f5526bd50ecd42af296f5587b7fc231f", "production": "docker", "python-fuelclient_sha": "5657dbf06fddb74adb61e9668eb579a1c57d8af8", "astute_sha": "6d6ad68e0cde286d74ac7d52e21da4fc8dcbe9ab", "feature_groups": ["mirantis"], "release": "6.1", "fuelmain_sha": "b975019fabdb429c1869047df18dd792d2163ecc", "fuellib_sha": "8384b8ca4db84794fb21e287202f05e31f78841c"

Changed in fuel:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.