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

Научить расширение подключаться с возможностью передачи ns #11

Closed
Rebulus opened this issue Mar 31, 2016 · 11 comments
Assignees
Milestone

Comments

@Rebulus
Copy link
Collaborator

Rebulus commented Mar 31, 2016

При использовании noscript на сервере, он подключается как функция, вызов которой инициализирует новый экземпляр noscript.

Например

var NS = require('noscript');
var ns = NS();

Необходимо научить расширение принимать экземпляр noscript, который оно будет дополнять новыми возможностями.

@Rebulus Rebulus self-assigned this Mar 31, 2016
@Rebulus
Copy link
Collaborator Author

Rebulus commented Mar 31, 2016

@vitkarpov как думаешь, чем лучше собрать итоговый модуль: gulp, grunt, webpack

@vitkarpov
Copy link
Member

@Rebulus Можно по-хипстерки, а можно галпом :)

@vitkarpov
Copy link
Member

@Rebulus а это нам нужно или, в итоге, решили отказаться?

@Rebulus
Copy link
Collaborator Author

Rebulus commented Apr 22, 2016

В той реализации сервера, которую вы используете, нет, не надо. Но вообще, по хорошему, для серверного использования надо, т.к. инстанс ns там не известен.

@Rebulus
Copy link
Collaborator Author

Rebulus commented Apr 22, 2016

Если мы сейчас внесём эти изменения, вам придётся при переходе на эту версию изменять текущую логику работы noscript-react

@Rebulus
Copy link
Collaborator Author

Rebulus commented Apr 26, 2016

@vitkarpov так что будем с этим делать?

@vitkarpov
Copy link
Member

@Rebulus кажется, добро. сделаешь пул?

@Rebulus
Copy link
Collaborator Author

Rebulus commented May 12, 2016

да, вечером посмотрю и сделаю)

@Rebulus
Copy link
Collaborator Author

Rebulus commented May 12, 2016

но это несовместимые изменения с предыдущей версией использования

@Rebulus
Copy link
Collaborator Author

Rebulus commented May 12, 2016

#20

vitkarpov added a commit that referenced this issue May 17, 2016
[#11] Добавлена сборка  для node.js и браузера
@Rebulus
Copy link
Collaborator Author

Rebulus commented May 17, 2016

#22

@Rebulus Rebulus closed this as completed May 17, 2016
@Rebulus Rebulus added this to the 1.1 milestone May 17, 2016
Rebulus pushed a commit that referenced this issue Jun 27, 2016
[#11] Добавлена сборка  для node.js и браузера
@vitkarpov vitkarpov modified the milestones: 1.1, 1.1.0 Dec 30, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants