Uh oh!
There was an error while loading. Please reload this page.
feat: progress bar for next batch - #1631
Conversation
fb3424e to
dc70bbdCompareCo-authored-by: Urix <43704209+uri-99@users.noreply.github.com>
Co-authored-by: Urix <43704209+uri-99@users.noreply.github.com>
…1630) Co-authored-by: Urix <43704209+uri-99@users.noreply.github.com>
Co-authored-by: Uriel Mihura <43704209+uri-99@users.noreply.github.com>
Co-authored-by: Urix <43704209+uri-99@users.noreply.github.com>
de876d1 to
bc961c5CompareFixed by setting the preference in a cookie and load it from the server
Also removed target attr as it wasn't needed and was showing warnings when compling
Co-authored-by: Urix <43704209+uri-99@users.noreply.github.com> Co-authored-by: Marcos Nicolau <marcosnicolau@lambdaclass.com>
Co-authored-by: Urix <43704209+uri-99@users.noreply.github.com>
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
@Oppen here is a suggestion to avoid clip-path, use tailwind and smooth transitions that should do it: defrender(assigns)do~H"""<divclass="w-full relative weight-700 rounded-lg"><divclass="w-full bg-accent/20 rounded-2xl"><pclass="ml-2 text-center relative text-foreground font-bold z-10"><%=@label %></p></div><divclass="top-0 left-0 h-full bg-accent p-1 rounded-2xl absolute transition-all"style={"width: #{@progress}%"}></div></div>"""end |
MarcosNicolau
commented
Dec 20, 2024
Other than those comments, the update logic worked fantastic. |
Oppen
commented
Dec 20, 2024
I can try this, but I don't quite grasp what it does. Also, note the duplicate label is intentional because it remains centered and depending on which half of the bar you see it it needs different styling. |
2c8f1a9 to
e0aaba2CompareOppen
commented
Dec 20, 2024
Co-authored-by: Urix <43704209+uri-99@users.noreply.github.com>
Co-authored-by: nicolau <marcosnicolau2003@gmail.com> Co-authored-by: Marcos Nicolau <marcosnicolau@lambdaclass.com> Co-authored-by: Urix <43704209+uri-99@users.noreply.github.com> Co-authored-by: Marcos Nicolau <76252340+MarcosNicolau@users.noreply.github.com>
Co-authored-by: Uriel Mihura <43704209+uri-99@users.noreply.github.com>
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
uri-99
left a comment
There was a problem hiding this comment.
getting this when running the explorer:
Error: other error: %ArithmeticError{message: "bad argument in arithmetic expression"}
which leads to a 500: key :cost_per_proof_chart not found in: %{
uri-99
left a comment
There was a problem hiding this comment.
add a default value for the new var
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Co-authored-by: Urix <43704209+uri-99@users.noreply.github.com> Co-authored-by: Marcos Nicolau <76252340+MarcosNicolau@users.noreply.github.com> Co-authored-by: PatStiles <33334338+PatStiles@users.noreply.github.com> Co-authored-by: Avila Gastón <72628438+avilagaston9@users.noreply.github.com> Co-authored-by: Marcos Nicolau <marcosnicolau@lambdaclass.com> Co-authored-by: nicolau <marcosnicolau2003@gmail.com>
Co-authored-by: Urix <43704209+uri-99@users.noreply.github.com> Co-authored-by: Marcos Nicolau <76252340+MarcosNicolau@users.noreply.github.com> Co-authored-by: PatStiles <33334338+PatStiles@users.noreply.github.com> Co-authored-by: Avila Gastón <72628438+avilagaston9@users.noreply.github.com> Co-authored-by: Marcos Nicolau <marcosnicolau@lambdaclass.com> Co-authored-by: nicolau <marcosnicolau2003@gmail.com>
Co-authored-by: Urix <43704209+uri-99@users.noreply.github.com> Co-authored-by: Marcos Nicolau <76252340+MarcosNicolau@users.noreply.github.com> Co-authored-by: PatStiles <33334338+PatStiles@users.noreply.github.com> Co-authored-by: Avila Gastón <72628438+avilagaston9@users.noreply.github.com> Co-authored-by: Marcos Nicolau <marcosnicolau@lambdaclass.com> Co-authored-by: nicolau <marcosnicolau2003@gmail.com>

Important
This PR requires a new .env variable
Add a new progress bar with inline label telling how much time until the next scheduled batch is pushed.
Type of change
Please delete options that are not relevant.
Testing
You may also play with different timeouts by setting the env var
MAX_BATCH_WAIT_MINUTESto different numbers of minutes.Checklist
testnet, everything else tostaging