Skip to content
This repository has been archived by the owner on May 29, 2023. It is now read-only.

initial code #1

Merged
merged 2 commits into from
May 6, 2020
Merged

initial code #1

merged 2 commits into from
May 6, 2020

Conversation

waterplea
Copy link
Member

No description provided.

@waterplea waterplea requested a review from MarsiBarsi May 6, 2020 12:12
# More info: https://help.github.com/articles/about-codeowners/
#

* @waterplea
Copy link
Member

Choose a reason for hiding this comment

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

так если в либу будут приходить PRы, то я не буду дергаться на ревью автоматически

Copy link
Member Author

Choose a reason for hiding this comment

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

Ну мы, вроде, и не добавляли друг друга никогда в то, что сами делали. Давай добавлю.

Copy link
Member

Choose a reason for hiding this comment

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

я добавлял...

import {ClassProvider} from '@angular/core';
import {LOCAL_STORAGE} from '@ng-web-apis/common';

export class StorageMock implements Storage {
Copy link
Member

Choose a reason for hiding this comment

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

а мы прям настоящий сторейдж реализовали, чтобы проще было имплементировать и все по типам сошлось?

Copy link
Member Author

Choose a reason for hiding this comment

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

Ну это тупо Map, но вроде да

@@ -0,0 +1,27 @@
const NEVER = new Promise<any>(empty);

export function identity<T>(v: T): T {
Copy link
Member

Choose a reason for hiding this comment

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

а что сразу в common не добавил?
или не добавляем туда штук, которые непосредственно к ангуляру не имеют отношения?

Copy link
Member Author

Choose a reason for hiding this comment

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

Да, не хочу делать cdk из него

@@ -0,0 +1,6 @@
/** Stripped down Request from Express */
Copy link
Member

Choose a reason for hiding this comment

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

а Universal только с express работает?

Copy link
Member Author

Choose a reason for hiding this comment

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

Нет, вроде, но по дефолту его советуют вроде. Ну тут и не весь Request, если у тебя что-то другое, то можно подкрутить. Есть идеи как по-универсальнее это реализовать?

@waterplea waterplea merged commit 673e15d into master May 6, 2020
@waterplea waterplea deleted the initial branch January 28, 2022 13:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants