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

feat: user management table #1211

Draft
wants to merge 69 commits into
base: dev
Choose a base branch
from
Draft

feat: user management table #1211

wants to merge 69 commits into from

Conversation

russchristina
Copy link
Contributor

@russchristina russchristina commented Apr 10, 2024

This PR is in draft mode until I can refine my notes. I also need to consult with Joe about an endpoint for the data so that I can remove the stubs and do some clenaup.

Pull Request type

Please check the type of change your PR introduces:

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no API changes)
  • Build-related changes
  • Documentation content changes
  • Other (please describe):

What is the current behavior?

Issue Number: IN-857

What is the new behavior?

Does this introduce a breaking change?

  • Yes
  • No

Other information

Summary by CodeRabbit

  • New Features

    • Introduced a new Management component in the Admin section for enhanced user management capabilities.
    • Added a new route /admin/management for accessing the Management page directly.
    • Enhanced user permissions to include viewAllUsers for better access control in data management.
    • Implemented new backend procedures and schemas to support user data operations effectively.
  • Documentation

    • Added comprehensive stories and documentation for the ManagementTable component to aid in development and testing.
  • Refactor

    • Updated peer dependencies in package.json to ensure compatibility with specific versions of react and react-dom.

Copy link

vercel bot commented Apr 10, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
inreach-app ⬜️ Ignored (Inspect) Visit Preview May 20, 2024 1:52pm

@github-actions github-actions bot added 📦 ui 📦 app automerge Enable Kodiak auto-merge kodiak: merge.method = 'squash' Kodiak will squash merge this PR. labels Apr 10, 2024
Copy link

alwaysmeticulous bot commented Apr 10, 2024

🤖 No test run has been triggered as your Meticulous project has been deactivated (since you haven't viewed any test results in a while). Click here to reactivate.

Last updated for commit 85ddfe0. This comment will update as new commits are pushed.

Copy link
Contributor

github-actions bot commented Apr 10, 2024

📦 Next.js Bundle Analysis for @weareinreach/app

This analysis was generated by the Next.js Bundle Analysis action. 🤖

New Page Added

The following page was added to the bundle from the code in this PR:

Page Size (compressed) First Load % of Budget (575 KB)
/admin/management 117.77 KB 529.08 KB 92.01%

@JoeKarow JoeKarow added the new-feature New features or options. label Apr 11, 2024
@JoeKarow JoeKarow changed the title In 857 user management feat: user management table Apr 11, 2024
Copy link

relativeci bot commented Apr 11, 2024

#402 Bundle Size — 3.4MiB (+0.14%).

85ddfe0(current) vs f7667d1 dev#399(baseline)

Warning

Bundle contains 5 duplicate packages – View duplicate packages

Bundle metrics  Change 7 changes Regression 2 regressions
                 Current
#402
     Baseline
#399
Regression  Initial JS 3MiB(+0.14%) 3MiB
No change  Initial CSS 9.54KiB 9.54KiB
Change  Cache Invalidation 54.19% 68.78%
Change  Chunks 68(+1.49%) 67
Change  Assets 81(+1.25%) 80
Change  Modules 2004(+0.2%) 2000
Regression  Duplicate Modules 271(+0.37%) 270
Change  Duplicate Code 7.88%(+0.38%) 7.85%
No change  Packages 210 210
No change  Duplicate Packages 5 5
Bundle size by type  Change 1 change Regression 1 regression
                 Current
#402
     Baseline
#399
Regression  JS 3.28MiB (+0.14%) 3.28MiB
No change  Fonts 94.54KiB 94.54KiB
No change  Other 9.62KiB 9.62KiB
No change  CSS 9.54KiB 9.54KiB
No change  IMG 8.57KiB 8.57KiB

Bundle analysis reportBranch IN-857-user-managementProject dashboard

Copy link

socket-security bot commented May 7, 2024

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 0

Review Details

Configuration used: CodeRabbit UI
Review profile: CHILL

Commits Files that changed from the base of the PR and between a9230db and 84eef06.
Files selected for processing (1)
  • package.json (1 hunks)
Files skipped from review as they are similar to previous changes (1)
  • package.json

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 0

Review Details

Configuration used: CodeRabbit UI
Review profile: CHILL

Commits Files that changed from the base of the PR and between 84eef06 and d676d9b.
Files selected for processing (1)
  • package.json (1 hunks)
Files skipped from review as they are similar to previous changes (1)
  • package.json

Copy link

sonarcloud bot commented May 20, 2024

Quality Gate Passed Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📦 api 📦 app automerge Enable Kodiak auto-merge kodiak: merge.method = 'squash' Kodiak will squash merge this PR. new-feature New features or options. 📦 ui
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants