Skip to content

Configurable defaults - #116

Open
pupeno wants to merge 2 commits into
driverdan:masterfrom
carouselapps:configurable-defaults
Open

Configurable defaults#116
pupeno wants to merge 2 commits into
driverdan:masterfrom
carouselapps:configurable-defaults

Conversation

@pupeno

Copy link
Copy Markdown
Contributor

I'm working on universal JavaScript, that is, running the same JavaScript on the browser and the server for pre-rendering server side applications and on client applications it's normal to do xmlhttprequests with just a path. In this case this library assumes you'll be talking to http://localhost:80/, but that's not always the case, so I made it possible to set a default, to bootstrap an environment where the application can run unchanged.

I'll be publishing blog posts and libraries I developed around this idea at https://CarouselApps.com in case you are curious.

@atroche

Copy link
Copy Markdown

any word on this? seems really useful.

@pupeno

Copy link
Copy Markdown
ContributorAuthor

@atroche if you need it right now, I released a fork with that and a small bug fix:

https://www.npmjs.com/package/@pupeno/xmlhttprequest

@pupeno

Copy link
Copy Markdown
ContributorAuthor

I updated the branch to 1.8.0, it merged without any issues.

@pupeno

Copy link
Copy Markdown
ContributorAuthor

Well, at least no conflicts.

Sign up for freeto 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

@pupeno@atroche