Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

GracePeriod query with hardcoded database name #358

Closed
aodpi opened this issue Oct 5, 2017 · 3 comments
Closed

GracePeriod query with hardcoded database name #358

aodpi opened this issue Oct 5, 2017 · 3 comments
Assignees
@unaizorrilla
Labels
Close Requested

Comments

@aodpi
Copy link

aodpi commented Oct 5, 2017

eShopOnContainers/src/Services/Ordering/Ordering.API/Infrastructure/HostedServices/GracePeriodManagerService.cs

Line 74 in e8c8678

@"SELECT Id FROM [Microsoft.eShopOnContainers.Services.OrderingDb].[ordering].[orders]

What if the database is configured with different name, Souldn't the context be used instead to query the Id
@unaizorrilla
Copy link
Collaborator

unaizorrilla commented Oct 5, 2017 •
edited by CESARDELATORRE
Loading

Yep .. I try to solve this issue a soon as posible.

Thanks
Unai

@unaizorrilla unaizorrilla self-assigned this Oct 6, 2017
@SychevIgor
Copy link
Contributor

@CESARDELATORRE it's commited

Date: 2 days ago (2017-10-06 11:09:16 AM)
Commit hash: ec5aee5
Children: eb38172#gitext://gotocommit/eb38172abf7661f9435d0e6834cacb6cbf124bb1
Parent(s): 3c2efbf#gitext://gotocommit/3c2efbf6783d396d89f37190639cc8b55eb40fc1

@CESARDELATORRE
Copy link
Collaborator

Closing this.

@javakian javakian mentioned this issue Nov 4, 2022
Open
@AJKFreitas AJKFreitas mentioned this issue Nov 4, 2022
Open
@UbuntuEvangelist UbuntuEvangelist mentioned this issue Nov 5, 2022
Open
@saurabharch saurabharch mentioned this issue Nov 5, 2022
Open
@ClaytonOSouza ClaytonOSouza mentioned this issue Nov 5, 2022
Open
@jessehouwing jessehouwing mentioned this issue Nov 5, 2022
Closed
@StevenRRNI StevenRRNI mentioned this issue Nov 5, 2022
Open
@muradheydarov muradheydarov mentioned this issue Nov 5, 2022
Open
@terrorizer1980 terrorizer1980 mentioned this issue Nov 5, 2022
Open
@ori-snyk ori-snyk mentioned this issue Nov 5, 2022
Open
@akishaha akishaha mentioned this issue Nov 5, 2022
Open
@BTDevelop BTDevelop mentioned this issue Nov 5, 2022
Open
@javakian javakian mentioned this issue Dec 25, 2022
Open
@AJKFreitas AJKFreitas mentioned this issue Dec 25, 2022
Open
@akishaha akishaha mentioned this issue Dec 25, 2022
Open
@BTDevelop BTDevelop mentioned this issue Dec 25, 2022
Open
@UbuntuEvangelist UbuntuEvangelist mentioned this issue Dec 25, 2022
Open
@saurabharch saurabharch mentioned this issue Dec 26, 2022
Open
@ClaytonOSouza ClaytonOSouza mentioned this issue Dec 26, 2022
Open
@jessehouwing jessehouwing mentioned this issue Dec 26, 2022
Open
@StevenRRNI StevenRRNI mentioned this issue Dec 26, 2022
Open
@muradheydarov muradheydarov mentioned this issue Dec 26, 2022
Open
@terrorizer1980 terrorizer1980 mentioned this issue Dec 26, 2022
Open
@ori-snyk ori-snyk mentioned this issue Dec 26, 2022
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@unaizorrilla unaizorrilla

Labels
Close Requested
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@SychevIgor @unaizorrilla @CESARDELATORRE @aodpi