twenty icon indicating copy to clipboard operation
twenty copied to clipboard

Error: Field "position" is not defined in "WorkspaceMemberOrderByInput".

Open harshit078 opened this issue 1 year ago • 5 comments

[!Caution]

  • When accessing properites such as Accout owner, an error is thrown GraphQL Error "Field "position" is not defined by type "WorkspaceMemberOrderByInput"
  • This issue is seems related to issue #7426

Current behavior

https://github.com/user-attachments/assets/0e5566a7-5d92-4a65-8793-06dedbc386c8

harshit078 avatar Oct 04 '24 09:10 harshit078

One possible approach to this issue can be that since we are trying to access Position field from WorkspaceMemberOrderByInput in graphQL schema which doesn't have Position defined in it. We can add position?: InputMaybe<OrderByDirection>; and keep it optional which will not log errors on both Sentry and Ts console.

harshit078 avatar Oct 04 '24 09:10 harshit078

I could work on it, please assign this issue to me

Utzig26 avatar Oct 04 '24 12:10 Utzig26

@harshit078 good catch. WorkspaceMember is a system object. We used to have something hardcoded to disable the links, I think that's what we should do for now. In the future we might be able to display the system objects within the app, but that will require backend work (right now not all fields on WorkspaceMember can be edited without consequence)

FelixMalfait avatar Oct 18 '24 12:10 FelixMalfait

/award 1000 (for creating all the mobile issues + this one + all the PRs)

FelixMalfait avatar Oct 18 '24 12:10 FelixMalfait

Awarding harshit078: 1000 points 🕹️ Well done! Check out your new contribution on oss.gg/harshit078

oss-gg[bot] avatar Oct 18 '24 12:10 oss-gg[bot]

Looks like this has been fixed

FelixMalfait avatar Nov 29 '24 16:11 FelixMalfait