Skip to content
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

refactor/general #93

Merged
merged 29 commits into from
Jul 11, 2023
Merged

refactor/general #93

merged 29 commits into from
Jul 11, 2023

Conversation

0x4007
Copy link
Member

@0x4007 0x4007 commented Jul 10, 2023

Small miscellaneous issues that have been bothering me. Started with style enhancements, but realized that there's a lot of cleanup that this codebase needed.

I didn't test the other dApps inside because I'm not familiar with how to use them. Perhaps I could get some help on QA? What is the quick import JSON object I can use to test audits on Gnosis Chain? I believe I am missing valid API and RPC properties.

Highlights

  • ran prettier
  • organized the project in a sensible manner
    • separated files in app-specific directories (we have payments, audits, secrets etc)
    • broke apart huge functions into separate files
  • one globals.d.ts and tsconfig.json added
  • generated permit now includes network query parameter on command line
  • adjusted styles on the claims table in the payments dApp
  • removed draw.ts because it was broken in an earlier pull request, and due to render performance being poor
  • error toaster on payouts dApp is more expressive (i pass it errors directly)
  • table no longer uses position: absolute or position: fixed so header and footer behave as expected without overlapping table elements.
  • commit footer renders instantly
  • not a full fix for Misleading permit already claimed message #91 but will render an error saying that the permit is not claimable (it won't say if its been invalidated or claimed though)
  • hide claim button when not on web3 enabled wallet (only metamask supported now) or not logged in to a wallet.

localhost_8080__claim=eyJwZXJtaXQiOnsicGVybWl0dGVkIjp7InRva2VuIjoiMHg2QjE3NTQ3NEU4OTA5NEM0NERhOThiOTU0RWVkZUFDNDk1MjcxZDBGIiwiYW1vdW50IjoiMTAwMDAwMDAwMDAwMDAwMDAwMDAwMCJ9LCJub25jZSI6Ijc1ODIwMzUwMjQxMjg2NTE4MDEzMzMyNzAzMDYwODI0NDg3NDkzMjU3OT

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Jul 10, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 0e6c04c
Status: ✅  Deploy successful!
Preview URL: https://9265845f.ubiquity-bounty-claims.pages.dev
Branch Preview URL: https://enhancements-styles.ubiquity-bounty-claims.pages.dev

View logs

@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

@0x4007 0x4007 marked this pull request as ready for review July 10, 2023 18:56
@0x4007 0x4007 requested a review from rndquu as a code owner July 10, 2023 18:56
@0x4007 0x4007 changed the title Enhancements/styles refactor/general Jul 10, 2023
@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

@0x4007
Copy link
Member Author

0x4007 commented Jul 10, 2023

Please help me QA the other pages (when I loaded them they seemed fine) and merge asap. This modifies a lot of files so its best to merge it now that no other hunters are working on it. @rndquu

@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

@0x4007
Copy link
Member Author

0x4007 commented Jul 11, 2023

  • I just realized that the color for the timer bar on the toaster is not rendering.
  • opacity for header needs to be dimmed on light mode.
  • opacity too low for beneficiary name.

Need to fix.

imageimage

Background and body elements should span to fill sides in dark mode

image

@rndquu
Copy link
Member

rndquu commented Jul 11, 2023

What is the quick import JSON object I can use to test audits on Gnosis Chain?

Audit page doesn't support gnosis chain right now (#86)

@0x4007 0x4007 mentioned this pull request Jul 11, 2023
4 tasks
@0x4007 0x4007 merged commit 0118279 into development Jul 11, 2023
@0x4007 0x4007 deleted the enhancements/styles branch July 11, 2023 09:58
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