-
Notifications
You must be signed in to change notification settings - Fork 4
Error in datatype for the "City" field in "sign-up as admin". #889
Comments
It's a valid issue 👍 |
@Monal5031, yes I am working on it. Will make the PR as soon as possible. |
is the issue solved? if not can i take it up |
There is a PR for this issue if @khatryshikha isn't working on it then sure you can @cristomathew |
Hi! I'd like to work on this issue. |
Hi, I'd love to work on this issue! |
@cristomathew do you still want to work on this issue or should I assign it to someone else? |
@Monal5031 Hi, i would like work on this issue. |
@Monal5031 Sorry i requested a PR without the issue being assigned to me. Can i fix this issue ? |
Hello, I'd like to work on this issue, Can I fix it? |
Sorry @Aimaanhasan @BeKingsman asked before you :( |
@Monal5031 I have linked a pull request which will fix this issue. |
I want to work on this as part of OSH |
Is this issue available to work on? I am a First Time Contributor. |
@alivabasa assigning this to you. |
@sanchibansal340 Thanks a lot ! Since I am a beginner , I m having a doubt whether the file is this or not. Plz guide me |
@alivabasa Were you able to find the file? |
yes, just submitting. |
Changed the data type for city in line 105 from "pk" to "name"
@sanchibansal340 plz review and tell me if any changes is required. |
Will check @alivabasa |
Is this issue a duplicate of #1002 ? |
@rpattath the issue is already solved in another pull request. Therefore, Should the issue and linked pull request be closed? |
Description
While signing up as an admin then City i.e is "string" is typecasted as int somewhere in the code.
Mocks
Acceptance Criteria
Update [Required]
Definition of Done
Estimation
0.5 hours
The text was updated successfully, but these errors were encountered: