search

cortexrd / Knack-Toolkit-Library

Knack Toolkit Library
MIT License
26 stars 6 forks source link

_cfv not working for equation field #362

Closed gstark1234 closed 2 months ago

gstark1234 commented 2 months ago

I put _cfv=ROI difference, lt, 0, red in the field description back end, but it's not working. It works for a text field but not sure why it's not working for my equation field. https://www.loom.com/share/77bd8980325944ada2fe4eea09f9d477?sid=9d440cfa-5aec-4edf-ba9d-cff83b44841a

CSWinnall commented 2 months ago

Hi @gstark1234

Could you try the same _cfv without the $ prefix on the field please. I believe that is causing the issue. Can you let me know if this is the case?

Craig

gstark1234 commented 2 months ago

Thanks, Craig. I tried that but still not working:

_cfv without $ - Watch Video https://www.loom.com/share/f87b6ca983bf417e8c2f5e53d59b9c95 https://www.loom.com/share/f87b6ca983bf417e8c2f5e53d59b9c95?utm_medium=gif

On Sat, Sep 14, 2024 at 3:49 PM Craig Winnall @.***> wrote:

Hi @gstark1234 https://github.com/gstark1234

Could you try the same _cfv without the $ prefix on the field please. I believe that is causing the issue. Can you let me know if this is the case?

Craig

— Reply to this email directly, view it on GitHub https://github.com/cortexrd/Knack-Toolkit-Library/issues/362#issuecomment-2351126438, or unsubscribe https://github.com/notifications/unsubscribe-auth/BKDMEXXHGQH7UCFBYR3RTG3ZWSHNXAVCNFSM6AAAAABN4JTBV2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJRGEZDMNBTHA . You are receiving this because you were mentioned.Message ID: @.***>

CSWinnall commented 2 months ago

Ok I'll try and find the issue and get Back to you

gstark1234 commented 2 months ago

Thank you!

On Tue, Sep 17, 2024 at 4:28 PM Craig Winnall @.***> wrote:

Ok I'll try and find the issue and get Back to you

— Reply to this email directly, view it on GitHub https://github.com/cortexrd/Knack-Toolkit-Library/issues/362#issuecomment-2356860336, or unsubscribe https://github.com/notifications/unsubscribe-auth/BKDMEXXS56QS77RAFKZ56GTZXCGFVAVCNFSM6AAAAABN4JTBV2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJWHA3DAMZTGY . You are receiving this because you were mentioned.Message ID: @.***>

CSWinnall commented 2 months ago

Good Morning I have fixed this issue and pushed some code for Norm to check before it goes live.

Craig

cortexrd commented 2 months ago

Thanks @CSWinnall for the fix.

I've deployed in Beta for testing.

@gstark1234, can you try it please?

ND

gstark1234 commented 2 months ago

Thx for all of your help but it still doesn't seem to be working. Here's what's happening:

https://www.loom.com/share/6220b577087146f990cd792ce1c2b09b?sid=25fa4d93-dacb-4c13-884a-12ead199f9a0

On Wed, Sep 18, 2024 at 8:55 AM Cortex R&D Inc. @.***> wrote:

Thanks @CSWinnall https://github.com/CSWinnall for the fix.

I've deployed in Beta for testing.

@gstark1234 https://github.com/gstark1234, can you try it please?

ND

— Reply to this email directly, view it on GitHub https://github.com/cortexrd/Knack-Toolkit-Library/issues/362#issuecomment-2358395037, or unsubscribe https://github.com/notifications/unsubscribe-auth/BKDMEXSMMYZQDEFR2JU6E4DZXFZ4JAVCNFSM6AAAAABN4JTBV2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJYGM4TKMBTG4 . You are receiving this because you were mentioned.Message ID: @.***>

CSWinnall commented 2 months ago

Hi @gstark1234

The field name needs to match the field label on the front end (Unless you use _lbl) not the field in the data:

This should work: _cfv=ROI, lt, 0, red

Craig

gstark1234 commented 2 months ago

Still a no go:

_cfv follow up - Watch Video https://www.loom.com/share/5ed8d1200b104946abde79023124affa https://www.loom.com/share/5ed8d1200b104946abde79023124affa?utm_medium=gif

On Wed, Sep 18, 2024 at 10:09 AM Craig Winnall @.***> wrote:

Hi @gstark1234 https://github.com/gstark1234

The field name needs to match the field label on the front end (Unless you use _lbl) not the field in the data:

This should work: _cfv=ROI, lt, 0, red

Craig

— Reply to this email directly, view it on GitHub https://github.com/cortexrd/Knack-Toolkit-Library/issues/362#issuecomment-2358577689, or unsubscribe https://github.com/notifications/unsubscribe-auth/BKDMEXU4BDUCOHQEFNCYQXDZXGCSBAVCNFSM6AAAAABN4JTBV2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJYGU3TONRYHE . You are receiving this because you were mentioned.Message ID: @.***>

cortexrd commented 2 months ago

@gstark1234 Here's a cool KTL feature you should use to see field information (or scene/views).

  1. You need the Developer role assigned to you
  2. Hold down Ctrl+Shift (Cmd+Shift on Mac) and hover around to see the Dev Info Popup with all the information you need.

See this video for a demo: https://www.youtube.com/watch?v=s396vEBKT8Q

Much simpler than using Inspect from the browser.

Norm

cortexrd commented 2 months ago

@gstark1234, I'll try to reproduce on my side.

Or, if you can/want to give me access to your page, it would be faster. Or re-create the bug in a new page with fake data.

ND

gstark1234 commented 2 months ago

How would I give you access to my page?

On Wed, Sep 18, 2024 at 10:26 AM Cortex R&D Inc. @.***> wrote:

@gstark1234 https://github.com/gstark1234, I'll try to reproduce on my side.

Or, if you can/want to give me access to your page, it would be faster. Or re-create the bug in a new page with fake data.

ND

— Reply to this email directly, view it on GitHub https://github.com/cortexrd/Knack-Toolkit-Library/issues/362#issuecomment-2358627158, or unsubscribe https://github.com/notifications/unsubscribe-auth/BKDMEXUDNVANO52QXZ7OBHLZXGEQXAVCNFSM6AAAAABN4JTBV2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJYGYZDOMJVHA . You are receiving this because you were mentioned.Message ID: @.***>

CSWinnall commented 2 months ago

Thank you Norm

I'm not sure what is going on as I can't replicate it now it works my end with the same numbers, prefix, commas and an equation field.

Craig

gstark1234 commented 2 months ago

So weird, that's not working either when I hit CMD+SHIFT on my Mac:

CMD+SHIFT - Watch Video https://www.loom.com/share/5b98e6842b7446c8af520b7e57ccdc3f https://www.loom.com/share/5b98e6842b7446c8af520b7e57ccdc3f?utm_medium=gif

On Wed, Sep 18, 2024 at 10:22 AM Cortex R&D Inc. @.***> wrote:

@gstark1234 https://github.com/gstark1234 Here's a cool KTL feature you should use to see field information (or scene/views).

  1. You need the Developer role assigned to you
  2. Hold down Ctrl+Shift (Cmd+Shift on Mac) and hover around to see the Dev Info Popup with all the information you need.

See this video for a demo: https://www.youtube.com/watch?v=s396vEBKT8Q

Much simpler than using Inspect from the browser.

Norm

— Reply to this email directly, view it on GitHub https://github.com/cortexrd/Knack-Toolkit-Library/issues/362#issuecomment-2358617418, or unsubscribe https://github.com/notifications/unsubscribe-auth/BKDMEXSG3J7CRQKHFLLVTHTZXGECTAVCNFSM6AAAAABN4JTBV2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJYGYYTONBRHA . You are receiving this because you were mentioned.Message ID: @.***>

CSWinnall commented 2 months ago

So weird, that's not working either when I hit CMD+SHIFT on my Mac: CMD+SHIFT - Watch Video https://www.loom.com/share/5b98e6842b7446c8af520b7e57ccdc3f https://www.loom.com/share/5b98e6842b7446c8af520b7e57ccdc3f?utm_medium=gif … On Wed, Sep 18, 2024 at 10:22 AM Cortex R&D Inc. @.> wrote: @gstark1234 https://github.com/gstark1234 Here's a cool KTL feature you should use to see field information (or scene/views). 1. You need the Developer role assigned to you 2. Hold down Ctrl+Shift (Cmd+Shift on Mac) and hover around to see the Dev Info Popup with all the information you need. See this video for a demo: https://www.youtube.com/watch?v=s396vEBKT8Q Much simpler than using Inspect from the browser. Norm — Reply to this email directly, view it on GitHub <#362 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/BKDMEXSG3J7CRQKHFLLVTHTZXGECTAVCNFSM6AAAAABN4JTBV2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJYGYYTONBRHA . You are receiving this because you were mentioned.Message ID: @.>

It's control + shift on a Mac.

gstark1234 commented 2 months ago

I'm so sorry to be a pain. Just providing feedback on your awesome tookkit - nothing happens when I hit control+shift on my Mac while hovering over a field and being logged in with a Developer role.

On Wed, Sep 18, 2024 at 10:30 AM Craig Winnall @.***> wrote:

So weird, that's not working either when I hit CMD+SHIFT on my Mac: CMD+SHIFT - Watch Video https://www.loom.com/share/5b98e6842b7446c8af520b7e57ccdc3f https://www.loom.com/share/5b98e6842b7446c8af520b7e57ccdc3f?utm_medium=gif … <#m-931463738701786489> On Wed, Sep 18, 2024 at 10:22 AM Cortex R&D Inc. @.> wrote: @gstark1234 https://github.com/gstark1234 https://github.com/gstark1234 https://github.com/gstark1234 Here's a cool KTL feature you should use to see field information (or scene/views). 1. You need the Developer role assigned to you 2. Hold down Ctrl+Shift (Cmd+Shift on Mac) and hover around to see the Dev Info Popup with all the information you need. See this video for a demo: https://www.youtube.com/watch?v=s396vEBKT8Q https://www.youtube.com/watch?v=s396vEBKT8Q Much simpler than using Inspect from the browser. Norm — Reply to this email directly, view it on GitHub <#362 (comment) https://github.com/cortexrd/Knack-Toolkit-Library/issues/362#issuecomment-2358617418>, or unsubscribe https://github.com/notifications/unsubscribe-auth/BKDMEXSG3J7CRQKHFLLVTHTZXGECTAVCNFSM6AAAAABN4JTBV2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJYGYYTONBRHA https://github.com/notifications/unsubscribe-auth/BKDMEXSG3J7CRQKHFLLVTHTZXGECTAVCNFSM6AAAAABN4JTBV2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJYGYYTONBRHA . You are receiving this because you were mentioned.Message ID: @.>

It's control + shift on a Mac.

— Reply to this email directly, view it on GitHub https://github.com/cortexrd/Knack-Toolkit-Library/issues/362#issuecomment-2358636946, or unsubscribe https://github.com/notifications/unsubscribe-auth/BKDMEXXVVX7W2AQ3JBY4GOLZXGE7RAVCNFSM6AAAAABN4JTBV2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJYGYZTMOJUGY . You are receiving this because you were mentioned.Message ID: @.***>

CSWinnall commented 2 months ago

Your not a pain. It's fine. Feedback is good. If you let Norm have access to your front end he should be able to figure it out.

Craig

cortexrd commented 2 months ago
  1. I've replicated you exact same setup in my app and all is working well.
  2. I'll need access to you app (not builder). Send me the link and password by email at nd@ctrnd.com
  3. You're not a pain LOL!!!
gstark1234 commented 2 months ago

No problem. Norm...let me know how to give you access, OR I'm happy to do a Zoom.

On Wed, Sep 18, 2024 at 10:38 AM Craig Winnall @.***> wrote:

Your not a pain. It's fine. Feedback is good. If you let Norm have access to your front end he should be able to figure it out.

Craig

— Reply to this email directly, view it on GitHub https://github.com/cortexrd/Knack-Toolkit-Library/issues/362#issuecomment-2358659076, or unsubscribe https://github.com/notifications/unsubscribe-auth/BKDMEXUGMVUPC3KPELWQFFTZXGGAFAVCNFSM6AAAAABN4JTBV2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJYGY2TSMBXGY . You are receiving this because you were mentioned.Message ID: @.***>

cortexrd commented 2 months ago

Sure, I'm ready now.

gstark1234 commented 2 months ago

Great...here's the Zoom link: https://us02web.zoom.us/j/2032317579

On Wed, Sep 18, 2024 at 10:42 AM Cortex R&D Inc. @.***> wrote:

Sure, I'm ready now.

— Reply to this email directly, view it on GitHub https://github.com/cortexrd/Knack-Toolkit-Library/issues/362#issuecomment-2358667887, or unsubscribe https://github.com/notifications/unsubscribe-auth/BKDMEXSQRUF4BBBB23PN7FDZXGGM3AVCNFSM6AAAAABN4JTBV2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJYGY3DOOBYG4 . You are receiving this because you were mentioned.Message ID: @.***>

cortexrd commented 2 months ago

Gary,

Can you test the Beta code again? Do the force refresh trick as I explained. Here, all is working ok in your app, and I didn't change anything.

ND

gstark1234 commented 2 months ago

YES!! It's working now in beta!! Thank you so much!!

Based on what we discussed today, when the new version is pushed it will update automatically and I'll get the update.

This is GREAT...THANK YOU?!!

On Wed, Sep 18, 2024 at 12:48 PM Cortex R&D Inc. @.***> wrote:

Gary,

Can you test the Beta code again? Do the force refresh trick as I explained. Here, all is working ok in your app, and I didn't change anything.

ND

— Reply to this email directly, view it on GitHub https://github.com/cortexrd/Knack-Toolkit-Library/issues/362#issuecomment-2358961948, or unsubscribe https://github.com/notifications/unsubscribe-auth/BKDMEXUVQAGSJ5DUUQ2BCS3ZXGVFLAVCNFSM6AAAAABN4JTBV2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJYHE3DCOJUHA . You are receiving this because you were mentioned.Message ID: @.***>

gstark1234 commented 2 months ago

Norm,

Thanks for all of your help today! Since we manually changed the code to "true" vs "false" for the control shift tool, is there something I need to do on the configurator page to make sure it sticks?

On Wed, Sep 18, 2024 at 10:22 AM Cortex R&D Inc. @.***> wrote:

@gstark1234 https://github.com/gstark1234 Here's a cool KTL feature you should use to see field information (or scene/views).

  1. You need the Developer role assigned to you
  2. Hold down Ctrl+Shift (Cmd+Shift on Mac) and hover around to see the Dev Info Popup with all the information you need.

See this video for a demo: https://www.youtube.com/watch?v=s396vEBKT8Q

Much simpler than using Inspect from the browser.

Norm

— Reply to this email directly, view it on GitHub https://github.com/cortexrd/Knack-Toolkit-Library/issues/362#issuecomment-2358617418, or unsubscribe https://github.com/notifications/unsubscribe-auth/BKDMEXSG3J7CRQKHFLLVTHTZXGECTAVCNFSM6AAAAABN4JTBV2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJYGYYTONBRHA . You are receiving this because you were mentioned.Message ID: @.***>

cortexrd commented 2 months ago

I always recommend to synchronize your Configurator to reflect your actual settings in the Builder, just by principle. Since it's a one-way process (Configurator to Builder), it requires that you do this manually.

There's the flag:

image

Don't forget to save and/or export your settings!

ND

CSWinnall commented 2 months ago

YES!! It's working now in beta!! Thank you so much!!

Based on what we discussed today, when the new version is pushed it will update automatically and I'll get the update.

This is GREAT...THANK YOU?!!

On Wed, Sep 18, 2024 at 12:48 PM Cortex R&D Inc. @.***> wrote:

Gary,

Can you test the Beta code again? Do the force refresh trick as I explained. Here, all is working ok in your app, and I didn't change anything.

ND

— Reply to this email directly, view it on GitHub https://github.com/cortexrd/Knack-Toolkit-Library/issues/362#issuecomment-2358961948, or unsubscribe https://github.com/notifications/unsubscribe-auth/BKDMEXUVQAGSJ5DUUQ2BCS3ZXGVFLAVCNFSM6AAAAABN4JTBV2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJYHE3DCOJUHA . You are receiving this because you were mentioned.Message ID: @.***>

I'm really glad it's working for you now.