Skip to content

Fix broken issue. Path was null. #607

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

Fix broken issue. Path was null. #607

wants to merge 1 commit into from

Conversation

@sjuarezgx
Copy link
Contributor

@sjuarezgx sjuarezgx commented Jun 8, 2022

Issue:96483
Path was being set null and the controller could not be calculated.

@genexusbot
Copy link
Collaborator

genexusbot commented Jun 8, 2022

Cherry pick to beta success

@sjuarezgx sjuarezgx temporarily deployed to external-storage-tests June 8, 2022 22:02 Inactive
@sjuarezgx sjuarezgx requested a review from claudiamurialdo June 8, 2022 22:02
@genexusbot
Copy link
Collaborator

genexusbot commented Jun 8, 2022

Revert to beta success

@claudiamurialdo
Copy link
Collaborator

claudiamurialdo commented Jun 8, 2022

Moved to #606

claudiamurialdo added a commit that referenced this pull request Jun 8, 2022
@claudiamurialdo
Add the fix made at #607 but keeping the conditional operator OR.
e4e32e1
claudiamurialdo added a commit that referenced this pull request Jun 9, 2022
@claudiamurialdo
Fix wrong path for api objects on Azure. Add unit test for that case. (…
c1c90c3 
…#606)

* Add new test for Azure httphandler and API object.

* Fix url casing for the test in order to reproduce error 500.

* Add the fix made at #607 but keeping the conditional operator OR.
claudiamurialdo added a commit that referenced this pull request Jun 9, 2022
@claudiamurialdo
Fix wrong path for api objects on Azure. Add unit test for that case. (…
c4413cb 
…#606)

* Add new test for Azure httphandler and API object.

* Fix url casing for the test in order to reproduce error 500.

* Add the fix made at #607 but keeping the conditional operator OR.

(cherry picked from commit c1c90c3)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@claudiamurialdo claudiamurialdo Awaiting requested review from claudiamurialdo

Assignees

No one assigned

Labels

bot closed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@sjuarezgx @genexusbot @claudiamurialdo