Skip to content

Task-8#7

Open
YurikMurik wants to merge 1 commit intotask-7from
task-8
Open

Task-8#7
YurikMurik wants to merge 1 commit intotask-7from
task-8

Conversation

@YurikMurik
Copy link
Copy Markdown
Owner

  • Added fake BE with data base
  • Refactored services in according to backend integration plan
  • Realized pagination in home-page component
  • Changed a logic of search component
  • Delete/adding of courses are working correct now
  • Created interceptor for auth token checking

- Added fake BE with data base
- Refactored services in according to backend integration plan
- Realized pagination in home-page component
- Changed a logic of search component
- Delete/adding of courses are working correct now
- Created interceptor for auth token checking
Comment thread package.json
"ngx-pagination": "^5.0.0",
"rxjs": "~6.5.4",
"tslib": "^1.10.0",
"zone.js": "~0.10.2"
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

json-server было бы здорово установить локально)

})
export class AuthService {
private userInfo: UserInfo;
private userUrl: string = 'http://localhost:3000/users';
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

лучше бы в константу

this.homePageService.getCourses().subscribe(
data => this.courses = data
);
this.updateCoursesMessageService.coursesReceivedNotify.subscribe(
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

надо будет отписаться на дестрой. Но это больше по rxjs модулю

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