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

ObjectMetadata decorator have an imageIdentifier properties #3710

Open
Weiko opened this issue Jan 30, 2024 · 8 comments
Open

ObjectMetadata decorator have an imageIdentifier properties #3710

Weiko opened this issue Jan 30, 2024 · 8 comments
Labels
prio: med scope: backend Issues that are affecting the backend side only size: short type: chore

Comments

@Weiko
Copy link
Member

Weiko commented Jan 30, 2024

Scope & Context

Identifiers are used by some client components to represent a record.

Screenshot 2024-01-30 at 18 46 36

Those identifiers are mapped to existing field-metadata and the object-metadata table contains both labelIdentifierFieldMetadataId and imageIdentifierFieldMetadataId.
We want to add a new property to the ObjectMetadata decorator to define those fields in order to have those defined during standard objects creation

This could look like this.

@ObjectMetadata({
  namePlural: 'people',
  labelSingular: 'Person',
  labelPlural: 'People',
  description: 'A person',
  icon: 'IconUser',
  labelIdentifier: 'name',
  imageIdentifier: 'avatarUrl',
})

Technical inputs

TBD

See also: #7616 (comment)

@FelixMalfait
Copy link
Member

labelIdentifierStandardId was implemented but no labelIdentifierImageId

@FelixMalfait FelixMalfait changed the title [Flexible-Schema] ObjectMetadata decorator have a labelIdentifier and an imageIdentifier properties [Flexible-Schema] ObjectMetadata decorator have an imageIdentifier properties Aug 30, 2024
@FelixMalfait FelixMalfait changed the title [Flexible-Schema] ObjectMetadata decorator have an imageIdentifier properties ObjectMetadata decorator have an imageIdentifier properties Sep 11, 2024
@charlesBochet charlesBochet added good first issue Good for newcomers scope: backend Issues that are affecting the backend side only size: short prio: med labels Sep 14, 2024
@Bonapara
Copy link
Member

Bonapara commented Oct 2, 2024

/oss.gg 150

Copy link

oss-gg bot commented Oct 2, 2024

Thanks for opening an issue! It's live on oss.gg!

@FelixMalfait FelixMalfait removed the good first issue Good for newcomers label Oct 2, 2024
@Aditya-PS-05
Copy link

/assign

Copy link

oss-gg bot commented Oct 10, 2024

Assigned to @Aditya-PS-05! Please open a draft PR linking this issue within 48h ⚠️ If we can't detect a PR from you linking this issue in 48h, you'll be unassigned automatically 🕹️ Excited to have you ship this 🚀

@sour2001
Copy link

/assign

Copy link

oss-gg bot commented Oct 10, 2024

This issue is already assigned to another person. Please find more issues here.

Copy link

oss-gg bot commented Oct 12, 2024

@Aditya-PS-05, Just a little reminder: Please open a draft PR linking this issue within 12 hours. If we can't detect a PR in 12h, you will be unassigned automatically.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
prio: med scope: backend Issues that are affecting the backend side only size: short type: chore
Projects
Status: 📋 Backlog
Development

Successfully merging a pull request may close this issue.

7 participants