Remove todo about GUID field on the 'aggregator_item' entity and add UUID field

Updated the test array to try and fix the errors in #32.

NW for the update paths. I started work on them, but haven't finished.

Looks good, just needs update path as you mentioned

I added the update path and test.

+++ b/tests/src/Functional/Update/AggregatorUpdateAddItemUuid.php
@@ -0,0 +1,34 @@
+  /**
+   * Ensure all items in the aggregator_feeds queue are deleted.
+   */
+  public function testUpdateHookN(): void {

I forgot to update this docblock when I copied it from another test I'd written. But I'm not going to bother changing it right now. We'll probably have to update this class's docblock to change the number of the update function it covers anyway. It can be fixed then.

Good to go, will need a reroll when one of the other update hook issues goes in, so fix the docblock at the same time

+++ b/aggregator.install
@@ -11,3 +11,15 @@
 function aggregator_update_last_removed() {
   return 8501;
...
+function aggregator_update_8601() {

both places will need to check and fix on commit

Rerolled #37

Thank you to everyone who worked on this!

Automatically closed - issue fixed for 2 weeks with no activity.

This issue was referenced in the jsonapi code to fix, not sure how this related to that. Silly that by now the module is removed from core, but appearantly the issue is still referenced by code.

core/modules/jsonapi/tests/src/Functional/ResourceTestBase.php:2067

Wonder if the code even gets triggered now?