-
Couldn't load subscription status.
- Fork 1.8k
fix: Choosing Ticket holder same as ticket buyer not makes the text light gray #4008
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
Conversation
Codecov Report
@@ Coverage Diff @@
## development #4008 +/- ##
==============================================
Coverage ? 21.82%
==============================================
Files ? 460
Lines ? 4737
Branches ? 0
==============================================
Hits ? 1034
Misses ? 3703
Partials ? 0
Continue to review full report at Codecov.
|
app/controllers/public/index.js
Outdated
| await order.save() | ||
| .then(order => { | ||
| this.notify.success(this.l10n.t('Order details saved. Please fill further details within 10 minutes.')); | ||
| this.notify.success(this.l10n.t('Order details saved. Please fill further details within 15 minutes.')); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Revert
|
Actually this is a wrong msg, it should be 15 minutes , that's why i changed it to 15 in this pr ,as it is a very small change. |
|
Even 15 minutes is wrong message.
…On Thu, 6 Feb, 2020, 10:20 Sundaram Dubey, ***@***.***> wrote:
Actually this is a wrong msg, it should be 15 minutes , that's why i
changed it to 15 in this pr ,as it is a very small change.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
<#4008?email_source=notifications&email_token=AKQMTLV4WDYF2MKTCB26N7LRBOJHVA5CNFSM4KQIZWO2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEK55POI#issuecomment-582735801>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AKQMTLSKAD4EXEGGUJ6UIBDRBOJHVANCNFSM4KQIZWOQ>
.
|
|
Should i change this to model.expiry-time something like this. |
|
Yes
…On Thu, 6 Feb, 2020, 10:49 Sundaram Dubey, ***@***.***> wrote:
Should i change this to model.expiry-time something like this.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
<#4008?email_source=notifications&email_token=AKQMTLWWKC54LUR4YUS3L3LRBOMUHA5CNFSM4KQIZWO2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEK565MY#issuecomment-582741683>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AKQMTLSZJYYLJ4XG2HE6NYDRBOMUHANCNFSM4KQIZWOQ>
.
|

Fixes #4004


Checklist
developmentbranch.