Skip to content

fix(download buttons past paper): add animation counting wallet after donwload file - #1068

Merged
sanaderi merged 2 commits into
GamaEdtech:upgrade-v4from
alireza013013:fix/animation-counting-past-paper
Jul 22, 2026
Merged

fix(download buttons past paper): add animation counting wallet after donwload file#1068
sanaderi merged 2 commits into
GamaEdtech:upgrade-v4from
alireza013013:fix/animation-counting-past-paper

Conversation

@alireza013013

Copy link
Copy Markdown
Collaborator

Description

add animation counting wallet after donwload file in past paper

Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Other (please describe):

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings/errors
  • I have added tests that prove my fix is effective or that my feature works

@vercel

vercel Bot commented Jul 21, 2026

Copy link
Copy Markdown

@alireza013013 is attempting to deploy a commit to the GamaEdtech Team on Vercel.

A member of the Team first needs to authorize it.

:is-processing="isLoading || isProcessingPayment"
:user-balance="balance"
:amount-to-pay="PRICE_FILE"
:amount-to-pay="5"

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Why is amount-to-pay hardcoded to 5 when each file has its own
price? This could show an incorrect required amount for files
that do not cost 5 GEM. Since the selected price is already
stored in priceFile, should this use :amount-to-pay="priceFile"
instead?

@sanaderi
sanaderi merged commit 8f57485 into GamaEdtech:upgrade-v4 Jul 22, 2026
1 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants