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.

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

Already on GitHub? Sign in to your account

magicvariable not used in sendNotification string interpolation #26

Open
chrysaora-achlyos opened this issue Jul 27, 2024 · 1 comment
Labels
bug Something isn't working

Comments

@chrysaora-achlyos
Copy link

Describe the bug

The following code does not behave as in AppStore Jellycuts

import Shortcuts
askForInput(prompt: "What is your name?", type: Text, default: "", allowDecimal: false, allowNegative: false) >> userName
sendNotification(body: "Hello ${userName}", title: "Meet the Team!", sound: true)

In particular, it does not use value of userName. The notification merely says "Hello"

after compiling the code, signing the shortcut, and opening in Shortcuts, there is no vertical line connecting the actions

To Reproduce

Steps to reproduce the behavior:

  1. paste the code above into oj_chat_2.jelly
  2. run the command jelly oj_chat_2.jelly --export --out u_oj_chat_2.shortcut
  3. run the command shortcuts sign --mode anyone --input u_oj_chat_2.shortcut --output oj_chat_2.shortcut
  4. run the command open -a Shortcuts oj_chat_2.shortcut
  5. click the "Add Shortcut" button
  6. click to select the oj_chat_2 Shortcut in Shortcuts
  7. click to open the selected shortcut
  8. run the shortcut with play icon on top right of shortcut
  9. Type in your name
  10. click the "Done" button
  11. click "Allow" button to allow notifications from shortcut
  12. Observe the Notification does not have your name
  13. Observe there is no vertical line between the two actions

Expected behavior

If one type "foo" at the name, the notification should say "Hello foo"

Information

  • Device: [macBook Pro, 14-inch, 2021]
  • OS: [Sonoma 14.5]
  • Jellycuts Version [??? jelly command does not provide version info (/opt/homebrew/bin/jelly)]
  • Jailbroken? [No]

Additional context

This may have the same cause as issue #24

@chrysaora-achlyos
Copy link
Author

likely fix in chrysaora-achlyos@ea0675a
the attachmentsByRange key values were being miscalculated,
Need to solve some issues among forks before generating pull request

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant