vaadin / expense-manager-demo

Progressive Web App (PWA) demo using Vaadin components
https://vaadin.com/components
Apache License 2.0
360 stars 142 forks source link

Adding an expense receipt fails and buttons become unresponsive #100

Closed macjuan closed 6 years ago

macjuan commented 6 years ago

Started up Expense Manager, added it to iOS 11.3 home screen and started the app. Added an expense and tried to add a receipt. Instead of showing the receipt it just shows bordered label "Receipt". Save, Cancel nor Delete buttons work. See attached video. img_8370 trim

macjuan commented 6 years ago

Is this now fixed at https://expensemanager.demo.vaadin.com/ ?

tomivirkki commented 6 years ago

No, deploying to https://expensemanager.demo.vaadin.com/ is not in scope of this ticket. There are other fixes to be merged before we deploy the new version. The fix for this ticket is now in master so the ticket is considered done.

manolo commented 6 years ago

Tried to deploy but there are some issues in CI infra, they are working in a fix. I'll inform when the app is available

manolo commented 6 years ago

@macjuan Deployed version in pro server works and have the fix if you want to check it out https://expensemanager-pro.demo.vaadin.com/

The normal version (open sourced) deployment is still failing.

manolo commented 6 years ago

@macjuan Finally everything is working fine. Now infra is configured that any new change in master is reflected in the site. CI takes care of it.