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

2092 Add payroll form UI #2149

Merged
merged 45 commits into from
Jul 17, 2024
Merged

Conversation

Da-Colon
Copy link
Contributor

@Da-Colon Da-Colon commented Jul 16, 2024

Closes #2092

Notes

  • UI only, though it is hooked up to the form ✅
  • Frequency, Frequency Amount, Asset Selector Implemented and connected to form, with some error handling ✅
  • Date Picker Not implemented ❌
  • Error Handling for payroll + roleInfo not fully implemented ❌
  • "Has Updated" Logic not implemented ❌ for payroll
  • Need @decentdao/design help (mentioned in standup, haven't looked in designs as of writing this for updates)
    • Tooltips around sections
    • External Links "Learn More" links

Other callouts

  • @decentdao/design The Drawer for Asset background is different from the 'frequency' selection, intentional?
  • @decentdao/engineering You'll notice the hardcoded values. because this is a menu, I was unable to use % values. I tried for a while different ways to be more responsive, I submit that I failed and maybe we kick this bucket down the road. as they look fine I think for most screens
    Desktop
    localhost_8888_roles_edit_dao=sep_0xd6Cb633E3A9cDEbA8a7A5CA4F30436F10FEbd418(Desktop)

Mobile
localhost_8888_roles_edit_details_dao=sep_0xd6Cb633E3A9cDEbA8a7A5CA4F30436F10FEbd418 hatId=0x0000016300010001000000000000000000000000000000000000000000000000(Mobile)

Da-Colon and others added 30 commits July 10, 2024 15:23
`2127` Disable submit proposal when submitting
…n-fix

`2128` ensure Role Info form validates correctly
…posal-ui

`2139` Clean up Create Role Proposal UI
@Da-Colon Da-Colon requested review from DarksightKellar, mudrila and a team July 16, 2024 01:52
@Da-Colon Da-Colon self-assigned this Jul 16, 2024
Copy link

netlify bot commented Jul 16, 2024

Deploy Preview for decent-interface-dev ready!

Name Link
🔨 Latest commit
🔍 Latest deploy log https://app.netlify.com/sites/decent-interface-dev/deploys/6697382c4901e633dd54ba9b
😎 Deploy Preview https://deploy-preview-2149.app.dev.decentdao.org
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@tomstuart123
Copy link

Looks sick @Da-Colon

To confirm, is the 'select asset' meant to be functional with what assets are in the treasury? Reason I ask is I don't see any assets appearing on mine (see screenshot) despite holding a token called $BUG. If that is intended behaviour, it could be because my treasury page is erroring again in this PR (not in production) so that could be the cause
Screenshot 2024-07-16 at 11 58 07

Another smaller thing, the drop down for 'frequency' drops downwards in mobile view but upwards in desktop view. See below
Screenshot 2024-07-16 at 11 58 26
Screenshot 2024-07-16 at 11 58 34

@Da-Colon
Copy link
Contributor Author

Looks sick @Da-Colon

To confirm, is the 'select asset' meant to be functional with what assets are in the treasury? Reason I ask is I don't see any assets appearing on mine (see screenshot) despite holding a token called $BUG. If that is intended behaviour, it could be because my treasury page is erroring again in this PR (not in production) so that could be the cause Screenshot 2024-07-16 at 11 58 07

Another smaller thing, the drop down for 'frequency' drops downwards in mobile view but upwards in desktop view. See below Screenshot 2024-07-16 at 11 58 26 Screenshot 2024-07-16 at 11 58 34

image

Weird, Bug shows up for me

@Da-Colon
Copy link
Contributor Author

Looks sick @Da-Colon

To confirm, is the 'select asset' meant to be functional with what assets are in the treasury? Reason I ask is I don't see any assets appearing on mine (see screenshot) despite holding a token called $BUG. If that is intended behaviour, it could be because my treasury page is erroring again in this PR (not in production) so that could be the cause Screenshot 2024-07-16 at 11 58 07

Another smaller thing, the drop down for 'frequency' drops downwards in mobile view but upwards in desktop view. See below Screenshot 2024-07-16 at 11 58 26 Screenshot 2024-07-16 at 11 58 34

As far as the Mobile drawer opening down. This happens automatically based on the space on mobile. if there isn't enough viewing space it will open going up. Doesn't appear I can change that behavior.

@tomstuart123
Copy link

@Da-Colon - hmmm. Have no idea why but my asset still isn't appearing.

@nicolaus-sherrill / @xraystyle1980 could one of you check? See comments above for context and the DAO in question here - https://deploy-preview-2149.app.dev.decentdao.org/roles/edit/details?dao=sep:0xB748f1a172C624cA088F3cD87D8A2139bA30df1d&hatId=0x5f02026fe45f68c1bcfbecb8dfa9867c4ecb2f1fa76d474dd3b4c6423d6a36bc

@Da-Colon
Copy link
Contributor Author

@Da-Colon - hmmm. Have no idea why but my asset still isn't appearing.

@nicolaus-sherrill / @xraystyle1980 could one of you check? See comments above for context and the DAO in question here - https://deploy-preview-2149.app.dev.decentdao.org/roles/edit/details?dao=sep:0xB748f1a172C624cA088F3cD87D8A2139bA30df1d&hatId=0x5f02026fe45f68c1bcfbecb8dfa9867c4ecb2f1fa76d474dd3b4c6423d6a36bc

Pretty sure I was looking at the deploy-preview before, but its not showing up for me now either. @mudrila The Moralis (Netlify/Function) network call isn't returning any data on the deploy preview.

@adamgall
Copy link
Member

adamgall commented Jul 17, 2024 via email

@Da-Colon
Copy link
Contributor Author

Probably because our dev Moralis plan runs out of “credits” or whatever by like 9 or 10 am every day, until the next day. Maybe we can sub in the prod moralis keys on dev. That’ll very likely fix these issues we’re seeing on dev and deploy previews regarding assets not appearing. I did like the pattern of separating all third party services into “dev” and “prod” scoped keys, but meh it’s probably fine if we just use a single Moralis plan for dev and prod. David / Kirill / Kelvin feel free to copy the Moralis keys from Netlify prod into Netlify dev

On Tue, Jul 16, 2024 at 7:02 PM David Colon @.> wrote: @Da-Colon https://github.com/Da-Colon - hmmm. Have no idea why but my asset still isn't appearing. @nicolaus-sherrill https://github.com/nicolaus-sherrill / @xraystyle1980 https://github.com/xraystyle1980 could one of you check? See comments above for context and the DAO in question here - https://deploy-preview-2149.app.dev.decentdao.org/roles/edit/details?dao=sep:0xB748f1a172C624cA088F3cD87D8A2139bA30df1d&hatId=0x5f02026fe45f68c1bcfbecb8dfa9867c4ecb2f1fa76d474dd3b4c6423d6a36bc Pretty sure I was looking at the deploy-preview before, but its not showing up for me now either. @mudrila https://github.com/mudrila The Moralis (Netlify/Function) network call isn't returning any data on the deploy preview. — Reply to this email directly, view it on GitHub <#2149 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAFMS4KT2JGJCGVEQWRR4K3ZMWRAHAVCNFSM6AAAAABK5TNXUKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMZRHE2TKOBRGE . You are receiving this because you are on a team that was mentioned.Message ID: @.>

Updated keys on netlify, and redeployed @tomstuart123 Just opened your link and its showing your token again.

@tomstuart123
Copy link

nice, this now works. Approved

@tomstuart123
Copy link

tomstuart123 commented Jul 17, 2024

@decentdao/engineering - could one of you review and approve also

UPDATE - ignore above. Synced with Kelvin and he'll pick up the ❌'s above first

@mudrila mudrila changed the base branch from develop to roles-0.2.0/streams July 17, 2024 11:09
@Da-Colon Da-Colon merged commit d1af778 into roles-0.2.0/streams Jul 17, 2024
7 checks passed
@Da-Colon Da-Colon deleted the issue/2092-add-payroll-form-ui branch July 17, 2024 15:36
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.

[roles-0.1.0] Create UI for Payroll Form
4 participants