Skip to content
This repository has been archived by the owner on Dec 5, 2022. It is now read-only.

Include demo SPA repos in README #200

Merged
merged 1 commit into from
Nov 30, 2017
Merged

Include demo SPA repos in README #200

merged 1 commit into from
Nov 30, 2017

Conversation

tsnolan23
Copy link
Contributor

@tsnolan23 tsnolan23 commented Nov 27, 2017

The examples provided by default through Tailor are great to get started, but I think linking directly to the SPA examples in the README would be helpful to those looking to dive a bit deeper into more complex and practical usage of Tailor (similar to the techniques used by Zalando in production).

The second of the two example SPAs was built in response to a dependency handling issue (#196)

@vigneshshanmugam
Copy link
Collaborator

Thanks @tsnolan23 for the PR :)

@vigneshshanmugam
Copy link
Collaborator

👍

1 similar comment
@DeTeam
Copy link
Contributor

DeTeam commented Nov 30, 2017

👍

@DeTeam DeTeam merged commit e8c0297 into zalando:master Nov 30, 2017
@stevoPerisic
Copy link

@tsnolan23, @vigneshshanmugam how would one put together a SPA with the use of fragments? This is not very clear as SPAs navigate via routes and the HTML is rendered on the front-end not on the server. I hope you see my dilemma, if I change routes within a SPA and want to navigate to another fragment how would I execute that?

@tsnolan23 tsnolan23 deleted the readme-include-demo-repos branch November 30, 2017 14:01
@tsnolan23
Copy link
Contributor Author

@vigneshshanmugam I know this has been closed, but @stevoPerisic makes a fair point. I'm also curious on that topic.

In certain instances, you're essentially "forfeiting" the true SPA behavior in terms of routing, no? If the fragment uses say, react-router to navigate within itself, it works fine, but in a case where the page is completely changing, it's an entirely new HTTP request through skipper/tailor, to get the new template correct?

It's not like code splitting where it would just make an asynchronous call for the next fragment?

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.

4 participants