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
fix(TJS-61): testing javascript course completion certificate #1425
fix(TJS-61): testing javascript course completion certificate #1425
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
const paymentOptions = defaultPaymentOptions({ | ||
stripeProvider: StripeProvider({ | ||
stripeSecretKey: process.env.STRIPE_SECRET_TOKEN, | ||
apiVersion: '2020-08-27', | ||
}), | ||
}) |
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.
why is this being commented out?
this is the direction we're heading to support multi-provider payments.
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.
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.
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.
@jbranchaud I think that correct import from is '@skillrecordings/skill-api/src'
and not '@skillrecordings/skill-api/
. Can you confirm?
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.
pushed the fix
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.
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.
The import should have already been correct. I think it may be an issue of re-building skill-api
locally (cd packages/skill-api; pnpm dev
).
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.
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.
can't build on my side. Does it work on yours?
43c93d9
to
87bbb47
Compare
6a82307
to
0d9fab8
Compare
This does two things:
courseCompleted
calculation function so that "Download Certificate" appears after all lesson get completed now.About the person who reported about this issue - I think we should let him know he needs to rewatch last lessons in each of the modules to get them completed and get certificate download link eventually