OmniLayer / omniwallet

Omni Protocol Hybrid Web-Wallet
https://www.omniwallet.org
GNU Affero General Public License v3.0
327 stars 187 forks source link

Critical error with Angular Bootstrap Directives #1095

Open Nevtep opened 9 years ago

Nevtep commented 9 years ago

Every time we need to add a new directive from angular bootstrap an error related to the template appears in the console. This also currently happens on new wallet creation for the tooltip and progressbar, which won't work until user logs out and relogins, we have some hacks at the wallet overview page that load the templates correctly, but lead to dirty code.

I lost about 3 hours trying to get this right and can't afford the time right now, creating this issue to track this down, nail it, and remove the hacks. So we don't forget.

The problem is URL related, the directives try to access urls without a leading "/" like this: "template/tabs/tabset.html" if the leading "/" is inserted the template loads correctly, if not, either angularjs or nginx go wrong, and return the index page instead, causing the template content to be invalid and thus, throw and error.

Nevtep commented 9 years ago

I found a posible fix for this, apparently removing

and leaving only

seems to have cleared the errors away in testnet, and the directives still work