Skip to content

Memberpanel - #599

Merged
BGL-PV merged 11 commits into
volunteer-dashboard-mobile-refactorfrom
memberpanel
Sep 13, 2026
Merged

BGL-PV merged 11 commits into
volunteer-dashboard-mobile-refactorfrom
memberpanel

Conversation

@BGL-PV

@BGL-PV BGL-PV commented Sep 13, 2026

Copy link
Copy Markdown
Contributor

omni PR

zoe2276 and others added 9 commits September 12, 2026 15:31
* added multitext field and alises user field

* breathing room

* lint

* prettier

* working session with benj

removal moved to onclick for whole pill

styling changes

* lint
Bumps [next](https://github.com/vercel/next.js) from 15.5.21 to 15.5.24.
- [Release notes](https://github.com/vercel/next.js/releases)
- [Commits](vercel/next.js@v15.5.21...v15.5.24)

---
updated-dependencies:
- dependency-name: next
  dependency-version: 15.5.24
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: zoe | bito <zoe@zlynn.dev>
This reverts commit d8e474e.
This reverts commit 776795b.
@BGL-PV
BGL-PV requested a review from zoe2276 September 13, 2026 14:06
@vercel

vercel Bot commented Sep 13, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
pv-frontpage-redux Ready Ready Preview Sep 13, 2026 7:33pm UTC

Request Review

/>
</FormGroup>

<Link

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

watch your render nesting depth

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

done

color: rgba(241, 245, 249, 0.95);
background: rgba(15, 23, 42, 0.92);
font-size: 0.72rem;
font-weight: 560;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

this is not a valid font weight i dont think, im pretty sure they have to be evenly divisible by 100?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

done

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

done

Comment thread src/app/volunteer_dashboard/panels/members/page.tsx
Comment on lines +632 to +634
if (navUserId == null) {
return
}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

condense short circuit

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

done

Comment on lines +814 to +815
className={styles.emptyState}
style={{ color: '#ef4444' }}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

use cn with styles.errorState here

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

done

Comment on lines +818 to +820
{userQuery.error instanceof Error
? userQuery.error.message
: 'Unknown error'}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

parseErrorMessage

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

done

Comment on lines +181 to +183
{isRefetching ? (
<span className={styles.refetchingPill}>Loading...</span>
) : null}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

&& (:

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

done

@BGL-PV
BGL-PV merged commit 7a5ebfb into volunteer-dashboard-mobile-refactor Sep 13, 2026
5 checks passed
@BGL-PV
BGL-PV deleted the memberpanel branch September 13, 2026 19:46
BGL-PV added a commit that referenced this pull request Sep 13, 2026
* Establishing the foundation for review

* forgot the protected page part of foundation

* implementing base NavigationStack Component (#563)

* implementing base NavigationStack Component

* implementing folder structure for clarity of how it will work

* addressing electro's comments

* i forgot to change

* removing placeholders

* Implementing detailview  (#564)

* implementing the detail subcomponent

* updating to improve syntax

* Update panelHistory.ts

* addressing electro's comments

* removing panel history

* changes

* navigation button (#565)

* navigation button

* lint

* massively refactoring navigation button

* implementing bito feedback

* Implementing the sidebar (#566)

* base files

* implementing sidebar code

* implementing bitos suggestions

* small change

* teeny css update

* Update SelectionIndicator.module.css

* panel implementation (#592)

* panel implementation

* bitos changes

* dwad

* implementing home page (#593)

* implementing dashboard home page

* improvements based on bito

* l

* Revert "l"

This reverts commit 63ba56f.

* fixing build error

* making the changes

* fundrasing implementation (#595)

* fundrasing implementation

* formatting

* bitos suggestions

* push

---------

Co-authored-by: zoe2276 <zoe@zlynn.dev>

* Implemnting roles contributions and donors (#597)

* feat/292/aliases (#591)

* added multitext field and alises user field

* breathing room

* lint

* prettier

* working session with benj

removal moved to onclick for whole pill

styling changes

* lint

* implementing the dog shit

* fuck you shell

* push

* fix page.module.css

---------

Co-authored-by: zoe | bito <zoe@zlynn.dev>

* Permissions and positions implemntation (#598)

* Implementing P&P

* chain selectism

* bitos changes

* Memberpanel (#599)

* feat/292/aliases (#591)

* added multitext field and alises user field

* breathing room

* lint

* prettier

* working session with benj

removal moved to onclick for whole pill

styling changes

* lint

* Bump next from 15.5.21 to 15.5.24 (#589)

Bumps [next](https://github.com/vercel/next.js) from 15.5.21 to 15.5.24.
- [Release notes](https://github.com/vercel/next.js/releases)
- [Commits](vercel/next.js@v15.5.21...v15.5.24)

---
updated-dependencies:
- dependency-name: next
  dependency-version: 15.5.24
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: zoe | bito <zoe@zlynn.dev>

* admin version

* new memberview

* Revert "new memberview"

This reverts commit d8e474e.

* Reapply "new memberview"

This reverts commit 776795b.

* and this

* chain selectors and undoing a false &&

* bitos changes

* Update DonorView.tsx

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: zoe | bito <zoe@zlynn.dev>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* initial

* Revert "initial"

This reverts commit 81c0425.

* Endorsement hell (#601)

* initial

* endorsement implementation

* fghjkl (#602)

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: zoe2276 <zoe@zlynn.dev>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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