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.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

August 14, 2019 #227

Closed
zetlen opened this issue Aug 12, 2019 · 3 comments
Closed

August 14, 2019 #227

zetlen opened this issue Aug 12, 2019 · 3 comments
Labels
meeting notes Topic requests and notes from meetings

Comments

@zetlen
Copy link
Contributor

zetlen commented Aug 12, 2019 •
edited by buskamuza

Please add your topic as a comment to the issue. Use following format:
Topic description and link to PR, if any (duration in min)

Time: 10:30am CST/CDT

BlueJeans Meeting - https://bluejeans.com/7385677850

馃帴 Recording
@zetlen zetlen added the meeting notes Topic requests and notes from meetings label Aug 12, 2019
@zetlen
Copy link
Contributor Author

zetlen commented Aug 12, 2019

RoutableInterface proposal for PWA to route all URLs: #226 (15 min.)

@buskamuza
Copy link
Contributor

buskamuza commented Aug 14, 2019 •
edited

  • @kandy : how Magento will know what's the typename?
  • @paliarush : it may be hard to define what's the product type.
  • @buskamuza : this proposal requires additional proposal for necessary backend implementation
  • @kandy : why did we decide to handle routing on backend side? @zetlen : because we want merchant to be able to control routing.
    • @kandy : if a user starts navigation from root, client always knows route information. But it's not always a case, so it doesn't make sense to implement two alternative ways of loading a page.

@buskamuza
Copy link
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
meeting notes Topic requests and notes from meetings
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@zetlen @buskamuza