Skip to content

Conversation

@justin808
Copy link
Member

We did not need to specify libraryTarget if using global.MyComponent
rather than the expose syntax for server rendering.

We did not need to specify libraryTarget if using global.MyComponent
rather than the expose syntax for server rendering.
@justin808
Copy link
Member Author

@samnang, @alexfedoseev, @dylangrafmyre, @mapreal19, @josiasds FYI. I will merge with one LGTM.

@samnang
Copy link
Contributor

samnang commented Oct 5, 2015

LGTM.

justin808 added a commit that referenced this pull request Oct 5, 2015
Updated docs for libraryTarget info
@justin808 justin808 merged commit a1ba6b9 into master Oct 5, 2015
@alex35mil
Copy link
Member

LGTM.

@robwise robwise deleted the doc-updates branch October 30, 2015 07:30
AbanoubGhadban pushed a commit that referenced this pull request Sep 25, 2025
Important:
* All renderer config settings begin with `renderer_`
* Password will use the password in the URL if not provided separately
* URL will default to localhost:3800 if not provided (or is missing say
  from an ENV value).

Also:
* Use the correct error class
* Make sure empty password gets ignored
* Various cleanup of configuration and docs
* Provide setup for running tests without the VmRenderer
* Missing tests for spec/react_on_rails_pro for ci
* remove poltergeist
* fix circleci config
* remove rspec/retry
AbanoubGhadban pushed a commit that referenced this pull request Sep 26, 2025
Important:
* All renderer config settings begin with `renderer_`
* Password will use the password in the URL if not provided separately
* URL will default to localhost:3800 if not provided (or is missing say
  from an ENV value).

Also:
* Use the correct error class
* Make sure empty password gets ignored
* Various cleanup of configuration and docs
* Provide setup for running tests without the VmRenderer
* Missing tests for spec/react_on_rails_pro for ci
* remove poltergeist
* fix circleci config
* remove rspec/retry
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.

4 participants