Exception: user_guid be be sent when getting Update Markers
Sentry Issue: ENGINE-D
Exception: user_guid be be sent
File "/Core/Notification/UpdateMarkers/Repository.php", line 34, in Minds\Core\Notification\UpdateMarkers\Repository::getList
throw new \Exception('user_guid be be sent');
File "/Core/Notification/UpdateMarkers/Manager.php", line 28, in Minds\Core\Notification\UpdateMarkers\Manager::getList
return $this->repository->getList($opts);
File "/Core/Groups/Membership.php", line 649, in Minds\Core\Groups\Membership::getGroupsByMember
'limit' => 1000,
File "/Controllers/api/v1/groups.php", line 52, in Minds\Controllers\api\v1\groups::get
'offset' => (int) $_GET['offset'],
File "/Api/Factory.php", line 70, in Minds\Api\Factory::build
return $handler->$method($pages);
...
(3 additional frame(s) were not displayed)
We need to check to make sure that we check for an authenticated user before we request update markers
added scoped labels
changed weight to 2
added 1 deleted label
added scoped labels
assigned to @benhayward.ben
changed time estimate to 1h
added 1 deleted label and removed 1 deleted label
added scoped label
changed weight to 1
added scoped label and automatically removed label
mentioned in merge request !372 (closed)
added scoped label and automatically removed label
- Developer
Have spent an hour on this but the solution I'm not happy with. Will discuss tomorrow.
added 1h of time spent at 2019-10-16
removed label
added scoped label
added scoped label and automatically removed label
added scoped label and automatically removed label
added scoped label and automatically removed label
removed 1 deleted label
added 1 deleted label
removed 1 deleted label
added 1 deleted label
removed label
- Developer
This is going to take some lower level changes than just checking for the user at the start of the failing endpoint.
changed time estimate to 3h
- Developer
Will discuss through the week.
added 1 deleted label and removed 1 deleted label
unassigned @benhayward.ben
added 1 deleted label and removed 1 deleted label
assigned to @benhayward.ben
changed weight to 3
added scoped label
added scoped label
removed 1 deleted label
removed label
changed milestone to %Sprint::01/01 - Xerothermic Xenartha
added scoped label and automatically removed label
added scoped label
removed label
added scoped label
- Developer
This is going to require a discussion with @brianhatchet and @edgebal because we can add a check to the Factory, but I believe this is something that has existed previously and was removed.
added scoped label and automatically removed label
added scoped label and automatically removed label
changed the description
added scoped label and automatically removed label
added scoped label and automatically removed label
- Developer
@benhayward.ben I updated the case with the appropriate fix
added scoped label and automatically removed label
moved to front#2577