-
Notifications
You must be signed in to change notification settings - Fork 43
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
Fix#143: Enable ability to edit org if website or github isn't provided #150
Conversation
@VividhPandey003 try making the value equal to the expected value that might fix it, also if you can tell me the scope of the testing I can help you further |
Oh actually I'm working on it. Thanks for having a look. I'll let you know if I face any issues. |
@VividhPandey003 #1496 has been merged! |
yes I have made the changes. just passing the tests and i'll change the pr from draft to get reviewed |
All checks passed and tested on local, Clear for merging @kevkevinpal |
@VividhPandey003 I think after your recent commit Cypress is failing can you take a look? |
hmm that is weird. |
@kevkevinpal can you check this Cypress Test fail? |
Signed-off-by: Vividh Pandey <[email protected]>
rebased and squashed your commits to one 14b3c15 |
Thanks @kevkevinpal |
Tested and it works. Good job! Paid. |
Describe your changes
fix.mov
Issue ticket number and link
Closes #143
Also tests out 1496 by @Vayras
Type of change
Please delete options that are not relevant.
Checklist before requesting a review