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

Switching to ConvertFrom-Json2 #23

Merged
merged 3 commits into from Apr 13, 2016
Merged

Conversation

LiamLeane
Copy link
Contributor

OOB ConvertFrom-Json implementation does not allow for overriding JSON
maxlength. Default is easy to exceed with large issue lists.

Change adds a ConvertFrom-Json2 function which allows override of
maxlength, sets the default to Max(int) and replaces all calls to
ConvertFrom-Json with ConvertFrom-Json2

OOB ConvertFrom-Json implementation does not allow for overriding JSON
maxlength. Default is easy to exceed with large issue lists.

Change adds a ConvertFrom-Json2 function which allows override of
maxlength, sets the default to Max(int) and replaces all calls to
ConvertFrom-Json with ConvertFrom-Json2
Added an extra 2 ad the end of the function name, doh.
@LiamLeane
Copy link
Contributor Author

I probably should have run the test suite myself first, works fine with the Jira-Issue but breaks other functions. I blame Friday.

Just updating the code now to play nice. Another commit forthcoming.

A couple of the other functions rely on expanded types so replicating
expanded types functionality in PS.
@replicaJunction
Copy link
Collaborator

Hi! Sorry this is a bit delayed; I've had some life changes recently that have affected my ability to work on this module.

Your ConvertFrom-Json2 function is super cool (honestly, that was the biggest thing keeping me on PS 3.0; I didn't want to rewrite the JSON cmdlets). Thanks for correcting the Pester test as well.

I don't have the ability to test this live against a JIRA instance right now. Can you confirm that with your change, the module still seems to be performing correctly (creating / resolving / editing issues and such)?

@LiamLeane
Copy link
Contributor Author

Confirming all is working as expected against a live JIRA instance.

On Sun, Apr 3, 2016 at 8:58 PM, Joshua T notifications@github.com wrote:

Hi! Sorry this is a bit delayed; I've had some life changes recently that
have affected my ability to work on this.

Your ConvertFrom-Json2 function is super cool (honestly, that was the
biggest thing keeping me on PS 3.0; I didn't want to rewrite the JSON
cmdlets). Thanks for correcting the Pester test as well.

I don't have the ability to test this live against a JIRA instance right
now. Can you confirm that with your change, the module still seems to be
performing correctly (creating / resolving / editing issues and such)?


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/replicaJunction/PSJira/pull/23#issuecomment-205090481

Liam Leane

Email: lleane.work@gmail.com
Skype: lleane.work

@replicaJunction replicaJunction merged commit 3047be1 into AtlassianPS:dev Apr 13, 2016
@replicaJunction
Copy link
Collaborator

Great. Thanks for the contribution!

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.

None yet

2 participants