Skip to content

Support PropertyForm metadata #1747

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

Closed
anjorinjnr opened this issue May 22, 2019 · 2 comments
Closed

Support PropertyForm metadata #1747

anjorinjnr opened this issue May 22, 2019 · 2 comments
Labels

Comments

@anjorinjnr
Copy link

I have property that I'm trying to ignore in json response

  @property.array(OAuth, {
    json: {
      out: false
    }
  })
  oauths: OAuth[];

I set the PropertyDefinition as shown above, but it doesn't seem to be having any impact. I also set it on the class not nada.

@bajtos
Copy link
Member

bajtos commented May 24, 2019

I am afraid we don't support property forms yet.

@raymondfeng IIRC, it was you who introduced property forms like json: {out: false} to property definition metadata in LB4. Do you have any opinions on how to go about implementing them in current juggler or even better in @loopback/repository?

@bajtos bajtos changed the title PropertyForm is ignored Support PropertyForm metadata May 24, 2019
@bajtos bajtos added the feature label May 24, 2019
@bajtos
Copy link
Member

bajtos commented May 24, 2019

Actually, let's move the discussion to loopback-next, where we already have an issue tracking this missing feature: loopbackio/loopback-next#2336

@bajtos bajtos closed this as completed May 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants