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

[MSYS-829] Fix nil class error when profile not found on automate server #321

Merged

Conversation

NAshwini
Copy link
Contributor

Fix nil class error when profile not found on automate server.
As per issue: #301

Signed-off-by: NAshwini ashwini.nehate@msystechnologies.com

Signed-off-by: NAshwini <ashwini.nehate@msystechnologies.com>
@NAshwini NAshwini requested a review from a team June 26, 2018 11:52
@NAshwini NAshwini changed the title [MSYS-829] Fix nil class error when profile not found on automate server WIP: [MSYS-829] Fix nil class error when profile not found on automate server Jun 26, 2018
@NAshwini NAshwini changed the title WIP: [MSYS-829] Fix nil class error when profile not found on automate server [MSYS-829] Fix nil class error when profile not found on automate server Jun 26, 2018
@NAshwini NAshwini force-pushed the ashwini/fix_nil_class_error branch 26 times, most recently from c7d6bcc to c289e10 Compare June 30, 2018 10:54
@NAshwini NAshwini force-pushed the ashwini/fix_nil_class_error branch from c289e10 to 0dd2b2d Compare July 3, 2018 10:11
Signed-off-by: NAshwini <ashwini.nehate@msystechnologies.com>
@NAshwini NAshwini force-pushed the ashwini/fix_nil_class_error branch from 0dd2b2d to ecafa89 Compare July 3, 2018 10:26
@NAshwini
Copy link
Contributor Author

NAshwini commented Jul 5, 2018

@btm , @vasundhara : Please review the PR.

Copy link

@miah miah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @NAshwini

Copy link
Contributor

@jerryaldrichiii jerryaldrichiii left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, thanks @NAshwini!

@NAshwini
Copy link
Contributor Author

Thank you @miah and @jerryaldrichiii

@btm btm merged commit 8b93976 into chef-boneyard:master Jul 17, 2018
Vasu1105 pushed a commit to MsysTechnologiesllc/audit that referenced this pull request Jan 8, 2019
…ver (chef-boneyard#321)

* Fix nil class error when profile not found on automate server

Signed-off-by: NAshwini <ashwini.nehate@msystechnologies.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants