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

Add validators for CAP inputs and Billing info inputs. #4233

Merged
merged 2 commits into from Feb 28, 2017

Conversation

akurinnoy
Copy link
Contributor

What does this PR do?

This PR adds

  • custom input type directive to allow user to enter only digits;
  • custom input type directive to allow user to enter symbols used in city names;
  • directive to validate city names;

What issues does this PR fix or reference?

https://github.com/codenvy/saas/issues/150
https://github.com/codenvy/saas/issues/153

Changelog

  • UD add directives for custom input types for numbers and city names;
  • UD add city name validation directive;

Release Notes

N/A - bugfix

Docs PR

N/A - bugfix

Signed-off-by: Oleksii Kurinnyi okurinnyi@codenvy.com

* @element
*
* @description
* `che-type-city` defines an attribute for input tag which allows to enter only digits. This directive should be used along with city-name-validator.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wrong description "enter only digits"

* @author Oleksii Kurinnyi
*/

export class CheTypeCity extends CheInputType{
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

space between CheInputType and {

@ashumilova ashumilova added this to the 5.4.0 milestone Feb 27, 2017
@codenvy-ci
Copy link

@@ -17,6 +17,8 @@ import {CheReloadHref} from './reload-href/che-reload-href.directive';
import {CheFormatOutput} from './format-output/che-format-output.directive';
import {CheOnLongTouch} from './touch/che-on-long-touch.directive';
import {CheOnRightClick} from './click/che-on-right-click.directive';
import {CheTypeNumber} from "./input-type/input-number.directive";
import {CheTypeCity} from "./input-type/input-city.directive";
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

double quote

@codenvy-ci
Copy link

Build # 2053 - FAILED

Please check console output at https://ci.codenvycorp.com/job/che-pullrequests-build/2053/ to view the results.

@slemeur
Copy link
Contributor

slemeur commented Feb 28, 2017

Oleksii Kurinnyi added 2 commits February 28, 2017 10:36
Added
- custom input type directive to allow user to enter only digits;
- custom input type directive to allow user to enter symbols used in city names;
- directive to validate city names;

Signed-off-by: Oleksii Kurinnyi <okurinnyi@codenvy.com>
Signed-off-by: Oleksii Kurinnyi <okurinnyi@codenvy.com>
@akurinnoy akurinnoy merged commit 7fc0a76 into master Feb 28, 2017
@akurinnoy akurinnoy deleted the billing-info-validation branch March 1, 2017 09:40
@JamesDrummond JamesDrummond mentioned this pull request Mar 8, 2017
9 tasks
JPinkney pushed a commit to JPinkney/che that referenced this pull request Aug 17, 2017
)

* Add validators for CAP inputs and Billing info inputs.

Added
- custom input type directive to allow user to enter only digits;
- custom input type directive to allow user to enter symbols used in city names;
- directive to validate city names;

Signed-off-by: Oleksii Kurinnyi <okurinnyi@codenvy.com>

* add tests

Signed-off-by: Oleksii Kurinnyi <okurinnyi@codenvy.com>
skabashnyuk pushed a commit that referenced this pull request Jan 3, 2020
* Add validators for CAP inputs and Billing info inputs.

Added
- custom input type directive to allow user to enter only digits;
- custom input type directive to allow user to enter symbols used in city names;
- directive to validate city names;

Signed-off-by: Oleksii Kurinnyi <okurinnyi@codenvy.com>

* add tests

Signed-off-by: Oleksii Kurinnyi <okurinnyi@codenvy.com>
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

Successfully merging this pull request may close these issues.

None yet

6 participants