Skip to content

Aayush create new page error#4813

Merged
one-community merged 7 commits intodevelopmentfrom
aayush_create_new_page_error
Apr 27, 2026
Merged

Aayush create new page error#4813
one-community merged 7 commits intodevelopmentfrom
aayush_create_new_page_error

Conversation

@aayushshetty12
Copy link
Copy Markdown
Contributor

Description

Screenshot 2026-02-07 224340

Related PRS (if any):

#3064
Use development branch for backend.

Main changes explained:

  • Updated CreateNewFile.jsx
  • Updated src/actions/userManagement.js

How to test:

  1. check into current branch
  2. do npm install and ... to run this PR locally
  3. Clear site data/cache
  4. log as admin user
  5. go to /bmdashboard/AddNewTeam
  6. check if the flow is smooth and form is submitted successfully.
  7. form details will be consoled if submitted successfully

Note:
No api for tasks has been integrated or handled completely as per the related PR. Using dummy data for tasks

Screenshots or videos of changes:

Recording.2026-01-31.185642.mp4

@netlify
Copy link
Copy Markdown

netlify Bot commented Feb 8, 2026

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit cf50a87
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/69e408b7d9c9460008d1e4f4
😎 Deploy Preview https://deploy-preview-4813--highestgoodnetwork-dev.netlify.app
📱 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 project configuration.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Feb 8, 2026

@one-community one-community added the High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible label Apr 3, 2026
Copy link
Copy Markdown

@rohanrastogi311 rohanrastogi311 left a comment

Choose a reason for hiding this comment

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

Hi Aayush,

No error messages if I don't add a team member.
No success message if form goes through.

Well done otherwise.

Image

Copy link
Copy Markdown

@Anusha-Gali Anusha-Gali left a comment

Choose a reason for hiding this comment

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

Hi Aayush,

I have reviewed your PR locally and though the functionality works, it requires below fixes:

  • The page looks different then that shown in your video, please confirm if it has changed
  • Assume current functionality, there is no confirmation on submit
  • In dark mode the warning appear in white text compared to red text in light mode
Image Image Image Image Image Image

@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown

@rohanrastogi311 rohanrastogi311 left a comment

Choose a reason for hiding this comment

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

HI Aayush,

Well done with this update.

Image Image

@one-community
Copy link
Copy Markdown
Member

Thank you all, merging!

@one-community one-community merged commit a5c41e6 into development Apr 27, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants