Skip to content
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

Boards board #41

Merged
merged 13 commits into from
Mar 4, 2024
Merged

Boards board #41

merged 13 commits into from
Mar 4, 2024

Conversation

igorkedzierawski
Copy link
Collaborator

image

@szymonpoltorak szymonpoltorak self-requested a review February 19, 2024 19:19
{{ isHidden(sliceWrapper) ? "chevron_right" : "expand_more" }}
</mat-icon>
<div class="flex items-center space-x-1">
<img class="w-5 h-5 rounded-full" src="{{ slice.metadata.avatarUrl }}" />
Copy link
Owner

Choose a reason for hiding this comment

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

brak alt attribute

}
</div>
</div>

<mat-menu #groupingMenu="matMenu">
@for (taskGrouping of TASK_GROUPINGS; track taskGrouping) {
Copy link
Owner

Choose a reason for hiding this comment

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

błąd z typowaniem

taskid: "t4",
assigneeAvatar: "/assets/corn-icons/icon-72x72.png",
assigneeName: "Bob Johnson"
protected filterString = "";
Copy link
Owner

Choose a reason for hiding this comment

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

brak typowania

assigneeAvatar: "/assets/corn-icons/icon-72x72.png",
assigneeName: "Bob Johnson"
protected filterString = "";
protected taskGrouping = TaskGrouping.BY_ASSIGNEE;
Copy link
Owner

Choose a reason for hiding this comment

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

brak typowania

enum TaskGrouping {
Copy link
Owner

Choose a reason for hiding this comment

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

czemu ten enum jest wewnątrz klasy. Powinien być w @enums

<div class="flex items-center justify-between">
<span class="text-gray-300 text-sm">{{ task.taskid }}</span>
<change-assignee-menu [associatedTask]="task">
<img
Copy link
Owner

Choose a reason for hiding this comment

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

alt

imports: [
MatCardModule,
ChangeAssigneeMenuComponent,
],
templateUrl: './taskcard.component.html',
})
export class TaskcardComponent {
Copy link
Owner

Choose a reason for hiding this comment

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

TaskCard


@ViewChild('droplistInstance') private droplistInstance!: CdkDropList;

getDroplistInstance() {
Copy link
Owner

Choose a reason for hiding this comment

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

  • DropList
  • typ
    • Możesz wykorzystać typescriptowego get'a zmienną nazwiesz z prefixem _ i będzie to sensowniejsze

export class TasklistComponent {

@Input() tasks: Task[] = [];
@Input() siblingDroplistInstances: any[] = [];
Copy link
Owner

Choose a reason for hiding this comment

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

DropList,
Czemu any[]

return this.droplistInstance;
}

protected droppedHandler(event: CdkDragDrop<Task[]>) {
Copy link
Owner

Choose a reason for hiding this comment

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

type

import { Task } from '@core/interfaces/boards/board/task.interface';

@Injectable()
export class ModelService {
Copy link
Owner

Choose a reason for hiding this comment

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

return types

@@ -1,17 +1,17 @@
import { Injectable } from '@angular/core';
import { SliceComponent } from './slice.component';
import { SliceComponent } from '../../../../../pages/boards/board/slice/slice.component';
Copy link
Owner

Choose a reason for hiding this comment

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

to możesz zastąpić za pomocą @pages

@@ -0,0 +1,112 @@
import { Injectable, Input } from '@angular/core';
import { SliceDescriptor } from '../../../../../pages/boards/board/slice/slice_descriptor';
Copy link
Owner

Choose a reason for hiding this comment

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

to też @pages

@@ -3,15 +3,21 @@ import { CommonModule } from '@angular/common';
import { MatInputModule } from '@angular/material/input';
import { MatButtonModule } from '@angular/material/button';
import { MatMenuModule } from '@angular/material/menu';
import { SlicesModelService, TaskChangedColumnEvent, TaskChangedGroupEvent, TaskGrouper } from './slice/slices_model.service';
import { SliceService } from './slice/slice.service';
import { SlicesModelService } from '../../../core/services/boards/board/slice/slices_model.service';
Copy link
Owner

Choose a reason for hiding this comment

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

@core

import { SlicesModelService, TaskChangedColumnEvent, TaskChangedGroupEvent, TaskGrouper } from './slice/slices_model.service';
import { SliceService } from './slice/slice.service';
import { SlicesModelService } from '../../../core/services/boards/board/slice/slices_model.service';
import { SliceService } from '../../../core/services/boards/board/slice/slice.service';
Copy link
Owner

Choose a reason for hiding this comment

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

@core

import { CommonModule } from '@angular/common';
import { MatInputModule } from '@angular/material/input';
import { MatButtonModule } from '@angular/material/button';
import { MatDividerModule } from '@angular/material/divider';
import { ModelService } from '../model.service';
import { BoardModelService } from '../../../../core/services/boards/board/model.service';
Copy link
Owner

Choose a reason for hiding this comment

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

@@ -1,11 +1,11 @@
import { AfterViewInit, Component, Input, OnDestroy, ViewChild } from '@angular/core';
import { CommonModule } from '@angular/common';
import { TaskListComponent } from '../task_list/task_list.component';
import { SliceService } from './slice.service';
import { SliceService } from '../../../../core/services/boards/board/slice/slice.service';
Copy link
Owner

Choose a reason for hiding this comment

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

@szymonpoltorak szymonpoltorak merged commit c7f979d into devel Mar 4, 2024
1 check passed
@szymonpoltorak szymonpoltorak deleted the boards-board branch March 4, 2024 21:28
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