-
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
Test: Write test successful login user gets redirected to Homepage with username on main navbar #33
Comments
@meenakshi-dhanani Can I work on this issue? |
Hi! @mtreacy002 Can I work on this issue? |
sure @aastha985. Go ahead 👍. |
Hi! I had to update about the status in 3 days. Haven't been able to work on this yet. I will do it by the weekend. I hope that's fine. |
Yep, sure, @aastha985. Thank you for notifying us on your situation. Let me know if you need any help with the task. |
Sure! Thanks😊 |
Hi @aastha985 , I'll make this issue available again since you are now working on backend issue#70 |
Sure! @mtreacy002 |
Hi @mtreacy002, I would like to work on this but have no experience. Looking forward to learning something new. |
Sure, @rpattath , I'll assign you to this. Let me know if you are facing any issue. we can try work it out together 😉 |
Can this be worked? |
@rpattath , are you still working on this issue? Can you please give me an update? |
Update. This issue is now open to anyone that would like to work on it. |
@mtreacy002 can you please assign this issue to me. |
@mtreacy002 i would like to give it a go . |
No, I am not.
…On Thu, 25 Feb 2021, 04:13 Maya Treacy, ***@***.***> wrote:
@jalajcodes <https://github.com/jalajcodes> , since there's no update
from @emmanuelhashy <https://github.com/emmanuelhashy>, I'll make this
issue available again. Let me know if you're still interested (that's if
you're not currently working on other issues 😉).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#33 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AKIM6RMYQTE4AKTH7A7W3TDTAW563ANCNFSM4OJXWTLA>
.
|
@emmanuelhashy, can you give us an update on this then? |
@mtreacy002 Since this is available again, is this issue something doable for a first time contributor? There are currently no unassigned "first timer" issues available. |
@rkraeher, this issue might be a bit complicated for a first-timer. But if you have some experience in the tech stack and confident you can work on it, I can assign you to this. For their first contribution to the project, a contributor doesn't have to limit themselves to a So, let me know if you do want to work on this 😉 |
ok thank you. I have experience with React and jest. I looked at the Login.test file and briefly at the React Testing Library docs and I believe I can do it but I was wondering if I can get some clarification on the user story? Right now there is one test for a successful login testing that an error message doesn't show up (lines 29-46 in Login.test). Am I on the right track to begin something like:
|
You're on the right target, @rkraeher . Would you like me to assign you to this? |
great! yes can you please assign it to me? thanks! |
If I'm having some trouble setting up the environment should I ask you here or get some help in Zulip? |
Done, I've assigned you on this @rkraeher . Yep, if you are facing trouble setting up, please reach out to us. Zulip will be a better option for trouble related to setting up, since you'll reach a wider audience compared to here on GitHub. Others who might have experienced the same issue can suggest a solution based on their experience. The post will also be helpful to other contributors that also trying to set up their project. So, I always encourage people to ask questions on zulip for issues related to setting up 😉. |
@rkraeher , any update on this issue? |
Sorry for the delayed response. I wrote some of this on Zulip but with your help I managed to get the app running in the browser but there is now another missing module error that I need to figure out so I'm still working on getting my environment set up. |
missing module: psycopg2._psycopg |
@mtreacy002 Ok I did as you suggested, installed psycopg2 separately, and it is now working. I will now proceed to link up BIT with MS so I have both servers running. |
@rkraeher , any updates on this issue? |
@mtreacy002 I now have successfully set up both servers and have the frontend site up locally. Starting today I can begin working on writing the test. |
@rkraeher , any updates on this issue? |
Putting this on hold following discussion with the contributor on zulip |
hey there @mtreacy002. i'm feeling better and was interested in getting involved again on the project during hacktober. Can I get back to work on this issue? Or if there are other areas where contribution would be more helpful im flexible and open to suggestion :) |
@rkraeher welcome back 🎉 |
hi @vj-codes and @mtreacy002 I finally managed to open the PR for this. sorry for the delay. Tagging you both here since the zulip stream seems a bit stale. |
Description
As a developer,
I need test if redirection post successful login works
so that I can make sure user who has successfully logged-in gets redirected to the homepage with the main navbar showing username of login user.
Mocks
Acceptance Criteria
Update [Required]
Enhancement to Update [Optional]
NA
Definition of Done
Estimation
1 day
The text was updated successfully, but these errors were encountered: