New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
REACH2-704:Reach new dashboard in admin console #8805
Conversation
plugins/reach/lib/kReachManager.php
Outdated
@@ -500,6 +500,14 @@ public static function addEntryVendorTask(entry $entry, ReachProfile $reachProfi | |||
$entryVendorTask->setAccessKeyExpiry($accessKeyExpiry); | |||
$entryVendorTask->setAccessKey(kReachUtils::generateReachVendorKs($entryVendorTask->getEntryId(), $shouldModerateOutput, $accessKeyExpiry)); | |||
$entryVendorTask->setPrice(kReachUtils::calculateTaskPrice($entry, $vendorCatalogItem)); | |||
$turnAroundTime = $vendorCatalogItem->getTurnAroundTime(); | |||
if ($turnAroundTime == -1) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
consult product manager
plugins/reach/lib/kReachManager.php
Outdated
{ | ||
$turnAroundTime = 0; | ||
} | ||
$entryVendorTask->setExpectedFinishTime($turnAroundTime); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove , until it is not queue - value should be zero
$this->setQueueTime(time()); | ||
$time = time(); | ||
$this->setQueueTime($time); | ||
$this->setExpectedFinishTime($this->getExpectedFinishTime() + $time); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Get the expected time from the profile
|
||
protected function applyFiltersAccordingToPartner($filter) | ||
{ | ||
if (kCurrentContext::$ks_partner_id == -2) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use existing const from Partner::
release-notes.md
Outdated
rt_attr_timestamp=expected_finish_time | ||
|
||
### Deployment scripts ### | ||
- php deployment/updates/scripts/add_permissions/2019_03_05_add_reach_extendAccessKey_action.php |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add new script
No description provided.