BusinessCentral.LinterCop icon indicating copy to clipboard operation
BusinessCentral.LinterCop copied to clipboard

Suggestion: Remove fieldgroup is missing for Setup Tables

Open kevin-smet opened this issue 2 years ago • 2 comments

With the latest update there is a "Fieldgroup brick/dropdown is missing on table" info. I think that's a good addition. But for setup tables this "requirement" should be removed (I think), as it doesn't make sense. It would be easy to check if you could define a "type" for your table. Now you can only do this based on the name of a table. Maybe you have other suggestions?

kevin-smet avatar Nov 14 '22 09:11 kevin-smet

That makes sense, but I think I can just tie it the „Primary Key“ field

/Stefan


Von: Kevin Smet @.> Gesendet: Monday, November 14, 2022 10:43:29 AM An: StefanMaron/BusinessCentral.LinterCop @.> Cc: Subscribed @.***> Betreff: [StefanMaron/BusinessCentral.LinterCop] Suggestion: Remove fieldgroup is missing for Setup Tables (Issue #241)

With the latest update there is a "Fieldgroup brick/dropdown is missing on table" info. I think that's a good addition. But for setup tables this "requirement" should be removed (I think), as it doesn't make sense. It would be easy to check if you could define a "type" for your table. Now you can only do this based on the name of a table. Maybe you have other suggestions?

— Reply to this email directly, view it on GitHubhttps://github.com/StefanMaron/BusinessCentral.LinterCop/issues/241, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AF5QBJ7X2YKREU4UFRYNUDTWIICUDANCNFSM6AAAAAAR7TT2TE. You are receiving this because you are subscribed to this thread.Message ID: @.***>

StefanMaron avatar Nov 14 '22 10:11 StefanMaron

Yep.. that would work.

kevin-smet avatar Nov 14 '22 10:11 kevin-smet

Hi @ShareTEC, I've created an PR for this. Could you review this PR whether it meets your expectations?

Arthurvdv avatar May 13 '23 14:05 Arthurvdv