Addepar / Addepar/ember-table

Pass metaData to onSelect (and allow custom metaData properties)

Open
#807 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
JavaScript
Stars
1.7k
Forks
351
Avg merge
1d 22h
Merged PRs (30d)
1

Description

Please could the rowMeta be included in the array that is passed to onSelect?

Each of my row items has an id that I do not want to display to the user, but is needed for internal identification. My current solution is the use the body.rowMeta.index to cross reference it with my original rows data. However, this means that I cannot use the inbuilt selection, as onSelect does not pass the rowMeta and so I have no way of getting the id.

Following on from this, is would be really helpful if I could add my own items to rowMeta, so that I can access the id directly without having to look it up based on the index.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by tracing the existing onSelect callback and rowMeta handling in the table's selection flow. Confirm how selected rows are assembled and how row metadata is currently exposed. Done means onSelect includes each row's rowMeta and callers can provide and retrieve custom metadata such as an internal id.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
frontend
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.