CalderaWP / caldera-forms-pro

Do not use this if you have Caldera Forms 1.5.8 or later
https://CalderaForms.com/pro
3 stars 1 forks source link

In CF Pro tab, make not connected message useful. #56

Closed Shelob9 closed 7 years ago

Shelob9 commented 7 years ago

When you go to the tab and app connection isn't set -- no API keys-- it just says not connected.

It should have useful feedback and a link (in new tab) to settings screen.

Source:

I think You must add your API keys to use Caldera Forms Pro. and it's all a link is all we need.

BTW this isn't as easy as it looks because that file isn't generated with PHP so can't make a dynamic link there. Will have to do that in calderawp\calderaforms\pro\admin\scripts::data() which generates the data in strings index of state.

In a component method - this.$store.getters.strings() should get the strings, so if you add foo then let strings =this.$store.getters.strings(); this.foo = strings.foo;would set foo in component's scope for foo, presuming foo was declared indata()`

See: https://github.com/CalderaWP/caldera-forms-pro/commit/7226d5afb7acd50a1951d9c7664a8debf1bc69c8

This is for #43 and a blocker for merge of #55 which is and sorry about this, in the feature/53-43 branch. Create a PR against that please.

https://github.com/CalderaWP/caldera-forms-pro/blob/feature/53-43

Shelob9 commented 7 years ago

Close via e1c8d90