-
Notifications
You must be signed in to change notification settings - Fork 12
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/auto detect wrong network #664
base: develop
Are you sure you want to change the base?
Conversation
@@ -135,7 +135,7 @@ | |||
mask="#.##" | |||
fill-mask="0" | |||
reverse-fill-mask | |||
:rules="[() => (usdAmount < 3 ? 'Minimum allowed budget is 3 USD' : true)]" | |||
:rules="[() => (usdAmount < 0.01 ? 'Minimum allowed budget is 3 USD' : true)]" |
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.
Please set it back to 3 USD
@@ -302,12 +302,13 @@ async function onProceedPaymentDialog(props) { | |||
if (_currentPrompt.value) { | |||
$q.loading.show() | |||
if (!customWeb3modal.getAddress()) { | |||
$q.notify({ type: 'negative', message: 'Please connect your wallet and try again' }) | |||
$q.notify({ type: 'negative', message: ' please connect your wallet ' }) |
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.
Please use capital letters in the beggining of sentence :)
@@ -346,7 +346,7 @@ function handleDeleteEntry(entryId, promptId) { | |||
async function onProceedDepositFundDialog(props) { | |||
//let's check if the entry already have valid payment.. | |||
if (!customWeb3modal.getAddress()) { | |||
$q.notify({ type: 'negative', message: 'Please connect your wallet and try again' }) | |||
$q.notify({ type: 'negative', message: ' please connect your wallet ' }) |
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.
Please use capital letters in the beggining of sentence :)
const result = await getEventsForCampaign(advertise.campaignCode) | ||
|
||
if (result.status.includes('success')) { | ||
$q.notify({ message: 'events retreived successfully ', type: 'positive' }) |
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.
Please use capital letters in the beggining of sentence :)
await _completeAdvertise(advertise) | ||
} | ||
if (!customWeb3modal.getAddress()) { | ||
$q.notify({ type: 'negative', message: ' please connect your wallet ' }) |
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.
Please use capital letters in the beggining of sentence :)
$q.notify({ message: result?.error?.message, type: 'negative' }) | ||
const result = await claimPayment({ campaignCode: advertise.campaignCode, currentAmounSpentInMatic: currentAmountSpent.toString() }) | ||
if (result.status.includes('success')) { | ||
$q.notify({ message: 'campaign payment claimed successfully ', type: 'positive' }) |
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.
Please use capital letters in the beggining of sentence :)
//let's change the advertise status | ||
await _completeAdvertise(advertise) | ||
if (!customWeb3modal.getAddress()) { | ||
$q.notify({ type: 'negative', message: ' please connect your wallet' }) |
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.
Please use capital letters in the beggining of sentence :)
$q.notify({ message: result?.error?.message, type: 'negative' }) | ||
const result = await requestAndApproveWithdrawal({ campaignCode: advertise.campaignCode, currentAmounSpentInMatic: currentAmounSpent }) | ||
if (result.status.includes('success')) { | ||
$q.notify({ message: 'remaing budget withdrawn successfully ', type: 'positive' }) |
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.
Please use capital letters in the beggining of sentence :)
🛠 Description
detect wrong network.
correct advertisment flow.
Fixes # (issue)
✨ Type of change
🧠 Rationale behind the change
Why did you choose to make these changes? Were there any trade-offs you had to consider?
🧪 All Test Suites Passed?
📸 Screenshots (optional)
Please provide some screenshot for relevant changes
🏎 Quick checklist