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

update release notes for 2.1.0, add subVI to palette #48

Conversation

nate-moehring
Copy link
Contributor

Updated release notes and palette for PRs 39 and 41, improved error reporting.
image

@nate-moehring
Copy link
Contributor Author

Tested install of newly built package.
image

@nate-moehring
Copy link
Contributor Author

Built package
jki_lib_json_serialization-2.1.0.40.vip.zip

@nate-moehring
Copy link
Contributor Author

To be clear, I think the ball is in your park to review the built package when you get a chance.

@jimkring
Copy link
Contributor

jimkring commented Aug 17, 2024 via email

@jimkring
Copy link
Contributor

@nate-moehring looking great! I am making a few other minor tweaks and I'll just add them to this PR, if that's OK...

@jimkring
Copy link
Contributor

  1. I moved the utility VIs into subpalette
    image

  2. Made both error inputs of "Add Source Path if Error" required.
    image

  3. Made some edits to the VI Description (see above) -- started by explaining when to use this VI and why.

  4. renamed error in (no error) to simple error in (see above), since less is more (and it's a recommendation of dnatt I believe).

- Made both error inputs of "Add Source Path if Error" required.
- Made some edits to the VI Description -- started by explaining when to use this VI and why.
- renamed `error in (no error)` to simple `error in`.
@jimkring
Copy link
Contributor

  1. marked Get UTC Offset.vi as source only (seems we missed that one)

@jimkring
Copy link
Contributor

  1. renamed error in (no error) to no error for public VIs (only a few were affected)

@jimkring
Copy link
Contributor

  1. Added a new example VI that HTTP GET's some JSON and shows how to use the new error handling util.

image

@jimkring jimkring merged commit adb2c1d into JKISoftware:main Aug 18, 2024
@nate-moehring nate-moehring deleted the update-package-for-improved-error-reporting branch August 18, 2024 03:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants