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

"trim" and "lowercase" convertions and formatting properties not working #523

Closed
veekungx opened this issue Sep 1, 2014 · 7 comments
Closed

Comments

@veekungx
Copy link

veekungx commented Sep 1, 2014

{
"name": "Post",
"base": "PersistedModel",
"properties": {
"title": {
"type": "string",
"required": true,
"trim": true,
"lowercase": true
}

},
"validations": [],
"relations": {},
"acls": [],
"methods": []

}

the "trim" and "lowercase" on "title" properties not work when I post data to the API I got response with

{
"title": "SDFSFSDF ",
"id": "54049e7fb36b19c817a00012"
}

Did I do something wrong or misunderstand ?

@veekungx veekungx changed the title Conversion and formatting properties not working "trim" and "lowercase" convertions and formatting properties not working Sep 1, 2014
@fabien
Copy link
Contributor

fabien commented Sep 1, 2014

Unfortunately, this is currently not working - see also: loopbackio/loopback-datasource-juggler#128

@fabien fabien added bug and removed enhancement labels Sep 1, 2014
@bajtos bajtos added #review and removed #review labels Sep 6, 2014
@crandmck
Copy link
Contributor

@raymondfeng Should this be labeled as a doc issue? If so, can you clarify what needs to be changed or added to docs?

@OwenBrotherwood
Copy link
Contributor

@crandmck looks like this issue is lost in refactoring ...
I think what is being referenced to is http://docs.strongloop.com/display/public/LB/Model+definition+JSON+file (not sure).
As to if the code contains what is in the doc is "interesting" @raymondfeng would have to get back on the issue


Was this an issue for loopback or loopback-datasource-juggler ... Multi-Repo hell :(

@crandmck
Copy link
Contributor

From loopbackio/loopback-datasource-juggler#128, it looks like property and conversion options are not working / supported. @raymondfeng can you confirm? If that's true, I should remove that section from the docs (until it's fixed).

@BerkeleyTrue
Copy link
Contributor

This should definitely be removed

@crandmck
Copy link
Contributor

crandmck commented Aug 6, 2015

I "commented out" that section from http://docs.strongloop.com/display/LB/Model+definition+JSON+file, so it won't be visible to general viewers.

@crandmck
Copy link
Contributor

Going to close this since the issue is addressed. Will need to add it back once those features are actually implemented.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants