Default reference key isn't always a column

[qhello]

Expected behavior

By default, the reference key of a table should be setup to the table's primary key

Actual behaviour

On my local environment, the reference key of table Beneficiary was empty on the frontend, therefore it was defaulted to the Address field (I guess by alphabetical order).

This field is not a Column but instead is a ManyToOne relationship

Failure Logs

ValidationError: Unexpected field type: 'retailer.address' (found 'ManyToOne' expected 'Column')
    at Function.validate (/Users/qhello/src/sline/node_modules/@forestadmin/datasource-toolkit/src/validation/field.ts:19:15)
    at Function.validate (/Users/qhello/src/sline/node_modules/@forestadmin/datasource-toolkit/src/validation/field.ts:45:22)
    at /Users/qhello/src/sline/node_modules/@forestadmin/datasource-toolkit/src/validation/projection.ts:6:48
    at Array.forEach (<anonymous>)
    at Function.validate (/Users/qhello/src/sline/node_modules/@forestadmin/datasource-toolkit/src/validation/projection.ts:6:16)
    at Function.parseProjection (/Users/qhello/src/sline/node_modules/@forestadmin/agent/src/agent/utils/query-string.ts:58:27)
    at Function.parseProjectionWithPks (/Users/qhello/src/sline/node_modules/@forestadmin/agent/src/agent/utils/query-string.ts:69:5)
    at ListRoute.handleList (/Users/qhello/src/sline/node_modules/@forestadmin/agent/src/agent/routes/access/list.ts:22:25)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at ErrorHandling.errorHandler (/Users/qhello/src/sline/node_modules/@forestadmin/agent/src/agent/routes/system/error-handling.ts:17:7)
warning: [400] GET /beneficiary - 155ms

===== An exception was raised =====
GET /beneficiary?{
 timezone: Europe/Paris,
 fields[beneficiary]: createdAt,id,name,retailer,stripeAccountId,updatedAt,
 fields[retailer]: address,
 page[number]: 1,
 page[size]: 15,
 sort: -id
}

 Invalid projection 

Error: Invalid projection
    at Function.parseProjection (/Users/qhello/src/sline/node_modules/@forestadmin/agent/src/agent/utils/query-string.ts:63:6)
    at Function.parseProjectionWithPks (/Users/qhello/src/sline/node_modules/@forestadmin/agent/src/agent/utils/query-string.ts:69:5)
    at ListRoute.handleList (/Users/qhello/src/sline/node_modules/@forestadmin/agent/src/agent/routes/access/list.ts:22:25)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at ErrorHandling.errorHandler (/Users/qhello/src/sline/node_modules/@forestadmin/agent/src/agent/routes/system/error-handling.ts:17:7)
    at Logger.logger (/Users/qhello/src/sline/node_modules/@forestadmin/agent/src/agent/routes/system/logger.ts:20:7)
    at bodyParser (/Users/qhello/src/sline/node_modules/koa-bodyparser/index.js:95:5)
    at cors (/Users/qhello/src/sline/node_modules/@koa/cors/index.js:107:16)
===================================

Context

• Package Version:

"@forestadmin/agent@npm:beta":
  version: 1.0.0-beta.37
"@forestadmin/datasource-sql@npm:beta":
  version: 1.0.0-beta.32

[Scra3]

Hello,
Thank you very much for your feedback. 🥳
We will try to reproduce your issue. We will get back to you as soon as possible.

[Scra3]

Indeed your projection is not valid. Please, could you provide your .forestadmin-schema.json ? Also, please can you tell me the content of reference field list on the retailer collection ?

[qhello]

Hi @Scra3,

Thank you for your answer, here's the informations you've requested:

• reference field on retailer collection is set to none by default - which causes the issue.
  
  if I manually change the reference field to id for instance, everything works again.

• .forestadmin-schema.json -> https://gist.github.com/qhello/b3bddbdc4cccee97c0695617858a4726

[Scra3]

Thanks to your issue we have found a bug in the front part. The pull request is in review and will be merged in the next hours.
Thanks you.
Have good time to code.

[qhello]

Amazing - looking forward for the fix 😄

Have a great day