Skip to content

Comments

feat: corp projects#719

Open
Crypta-Eve wants to merge 2 commits intomasterfrom
corp_projects
Open

feat: corp projects#719
Crypta-Eve wants to merge 2 commits intomasterfrom
corp_projects

Conversation

@Crypta-Eve
Copy link
Member

Corp Projects! Finally!
image
image

Its not perfect. And needs polish on UI for the configurations.. But its a start.

Requires: eveseat/eveapi#445

->first();
$config = $this->normalizeProjectConfiguration($project->configuration);

// dd($project, $corporation);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this can be removed

})
->addColumn('reward', function ($raw) use (&$rawColumns) {
if (is_null($raw->reward_initial) || $raw->reward_initial == 0) {
return 'No Reward'; // TODO localise
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TODO that can be resolved

Comment on lines +109 to +114
['data' => 'name', 'title' => trans_choice('web::seat.name', 1)],
['data' => 'last_modified', 'title' => trans_choice('web::seat.last_modified', 1)],
['data' => 'state', 'title' => trans('web::seat.state')],
['data' => 'progress', 'title' => trans('web::seat.progress')],
['data' => 'reward', 'title' => trans('web::seat.reward')],
['data' => 'contributors_count', 'title' => trans_choice('web::seat.contributor', 2)],
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does sorting and searching work on all columns? I don't see it disabled on any column and usually you have to disable a few.

<h4>About</h4>

<div class="media">
<!-- TODO LATER: consider icons here -->
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm just marking all todos

<div class="row">
<div class="col-4">
<dl>
<dt>Career</dt>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this could probably be localized? There are also other instances in this file

<div class="modal-dialog modal-lg modal-dialog-scrollable">
<div class="modal-content">
<div class="modal-header bg-blue">
<h4 class="modal-title">Details</h4>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

localization

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants