-
Notifications
You must be signed in to change notification settings - Fork 93
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: Create and Update program page #79
Feat: Create and Update program page #79
Conversation
Update @meenakshi-dhanani . Here's the PR for Create and update program (there are 2 issues open for this so I put both on the description as I'm not sure which one to select). Here's the snapshot of the payload The strange thing is it doesn't matter whether I used parseFloat or not, it's still showing as string |
Update @meenakshi-dhanani . I've tried to put the data directly to backend swagger UI, and the error is on the |
f2112bc
to
0b28a6e
Compare
Update @anitab-org/bridgeintech-maintainers and @meenakshi-dhanani. I've managed to solve the issue with |
0b28a6e
to
48bd2fc
Compare
Update @meenakshi-dhanani and @anitab-org/bridgeintech-maintainers . I've confirm that the update in this PR already working. This PR is now complete. Can you please review? Thanks |
eb87add
to
cd93227
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The changes made in this PR were tested locally. Following are the results:
-
Code review - Done
-
All possible responses were tested as below:
Expected Result: As a user, I shoudl be abe to create and update program as an organziation representative.
Actual Result: Received an errorTypeError: programs.map is not a function
.
@mtreacy002 Attached is the screenshot of the error, hmm do you know what causes the problem?
@foongminwong , can you tell me the steps you did to get to this error? starting from when you register the user who you've tested here. |
Yes, the steps are: |
It's bit hard to read the commit numbers like that, @foongminwong . Can you please do it again by running
|
@foongminwong , do you have time now to hop on hangout to troubleshoot the bug? |
Add pytz all_timezones list Remove default href to organization-portfolio on MyOrganization navbar Remove old Timezone enums from enums.js Add temporary default link to MyOrganization Navbar Add hard coded url as temporary solution to view pages Fix Navigation Fix typo programProfile To Program using MyOrganization > Portfolio > Program path Temp solution Path Path option for My Organization Use navlink instead of card as submenu on myorganization Remove console log and unused comments Disabled field for view other organization 's program
Implement datetime-local as date input field Successfully Create Program Update program and add maxLength field check
cd93227
to
24c976f
Compare
@foongminwong . How did you go with your end? Are you able to solve the npm issue? Don't forget to also pulled the latest ms-backend-server from my fork repo that has new commits that fix GET /users/{user_id}, which was what causing the bug on @meenakshi-dhanani side when she tested pr #76. Let me know if you want to troubleshoot together like this morning. 😉 |
Description
Add create and update Program page functionalities
Fixes #53, #78
Type of Change:
Code/Quality Assurance Only
How Has This Been Tested?
My Space
>My Organization
> on the bottom left of the page, clickGo to Programs
> click onAdd Program
buttonChecklist:
Code/Quality Assurance Only