[#51] Tables getting bulkier with time

[Bertverbeek4PS]

Based on issue #51 - Tables getting bulkier with time.
Added two tables to retention policy:

• table 82563 "ADLSE Deleted Record"
• table 82566 "ADLSE Run"

Also added field export because this is needed for the retention policy

[DuttaSoumya]

@Bertverbeek4PS, I finally had time to catch up on my pending activities after conclusion of Directions. I have made a few comments on this PR. Please reach out directly/ via Teams in case of any concerns.

[Bertverbeek4PS]

@Bertverbeek4PS, I finally had time to catch up on my pending activities after conclusion of Directions. I have made a few comments on this PR. Please reach out directly/ via Teams in case of any concerns.

I thought also that you were busy. No problem!
I have change it based on your comments! Thanks for reviewing this!! And the option of deleting records looks better.

[DuttaSoumya]

Thank you for making these changes. Could you please sync with the latest version on the main branch by:

git pull origin main

and resolving conflicts

[Bertverbeek4PS]

Thank you for making these changes. Could you please sync with the latest version on the main branch by:

git pull origin main

and resolving conflicts

Thanks for looking! I have changed all in my last commit. Also the sync. Though that I have already done that. But it wasn't

[DuttaSoumya]

I see more objects added to the review- API pages are showing up as new objects.

Did running git pull origin main not resolve the issues? Also, if you commit, you will find that it conflicts with the main branch.

Please book me in for a short 15 minute meeting if I can probably advise you on Git.

best regards
Dutta

[Bertverbeek4PS]

I see more objects added to the review- API pages are showing up as new objects.

Did running git pull origin main not resolve the issues? Also, if you commit, you will find that it conflicts with the main branch.

Please book me in for a short 15 minute meeting if I can probably advise you on Git.

best regards Dutta

@DuttaSoumya sorry there was something strange with my branche. Localy it was mixed up with another branche. Have fixed that now.

[pri-kise]

@Bertverbeek4PS sorry for jumping in so late, but shouln'd the Tables being add to the allowed tables on Company Initialization, too?

[Bertverbeek4PS]

@Bertverbeek4PS sorry for jumping in so late, but shouln'd the Tables being add to the allowed tables on Company Initialization, too?

Never to late :). That is indeed a scenario that can happen indeed thanks! @DuttaSoumya agree?

[DuttaSoumya]

A few easy to fix comments.

[Bertverbeek4PS]

@Bertverbeek4PS sorry for jumping in so late, but shouln'd the Tables being add to the allowed tables on Company Initialization, too?

Never to late :). That is indeed a scenario that can happen indeed thanks! @DuttaSoumya agree?

I have added this to codeunit 82569 "ADLSE Execution"
Not sure if that is the right place or maybe codeunit 82567 ADLSE

[DuttaSoumya]

One comment made. Meanwhile app.json has a new version number.