HousewareHQ / frontend-engineering-octernship

45 stars 54 forks source link

Regarding the submission of the project #34

Closed KevinA-cpu closed 1 year ago

KevinA-cpu commented 1 year ago

Hello Houseware team! after much research looking into issues/pull requests I was led to believe that I to submit the project I need to do 2 things:

  1. Push all my code to main
  2. Open a pull request on this repo to submit, host the project if possible

I believe that I'm missing something regarding the Github classroom, I went to the Github Classroom page and I can only create a new classroom, how can I see instructions on how to submit?

Thank you for reading and I hope this issue can be resolved soon!

narayan954 commented 1 year ago

@KevinA-cpu I'm pretty sure, point 2 was mentioned nowhere, instead it was mentioned something like, the auto generated pull request on your repo is the one where you task submission lies and should not be closed at all, evaluations shall be done on that only. Ps: You should be able to see changes and commits you made in the auto generated pr by github classroom bot

KevinA-cpu commented 1 year ago

thanks for your reply! point 2 was a combination of the pull requests I saw opened and one of the member of Houseware comment that hosting the project make you standout. I suppose I have done all I'm suppose to do.

narayan954 commented 1 year ago

thanks for your reply! point 2 was a combination of the pull requests I saw opened and one of the member of Houseware comment that hosting the project make you standout. I suppose I have done all I'm suppose to do.

best of luck mate! 🍀

whalewalker commented 1 year ago

@KevinA-cpu @narayan954, quick question, did I follow the correct process to submit my project? Let me know if I missed any steps.

  1. I forked repo is created to my repo
  2. I clone this repo and cd into the app directory
  3. I creates a branch and make all the changes and commit there
  4. I push to my branch
  5. Create a PR to main branch

Please what am i missing ?

DevYemi commented 1 year ago

@whalewalker You forked the repo ?

i'm not sure but i thought we were supposed to clone the project, and make changes to the auto generated feedback branch.

Making changes on the auto generated feedback branch to complete the task

whalewalker commented 1 year ago

@DevYemi, when i add the project from octernship website, the project was forked to my repo automatically. Do you think that's the one I should clone? Can you please guide me if I'm mistaken?

DevYemi commented 1 year ago

@whalewalker Yeah you should have cloned the repo that was forked automatically.

You should have cloned it and made all your code changes in the main branch, the feedback branch was meant to be used by the reviewer to ask questions and give feedback on the project.

Check out this issue on submission, one of the Houseware member explained the submission requirement in detail there

6

whalewalker commented 1 year ago

If I understand correctly, you're suggesting that after cloning the forked repository, all changes should be made on the main branch and a pull request (PR) should then be created to merge these changes into the feedback branch, is that correct, @DevYemi?

KevinA-cpu commented 1 year ago

here's the story from the start, when you sign up for Octernship there's a new repo automatically created for you, what you're supposed to do was clone that repo and work on the Feedback branch, however Houseware change their Instructions now you have to work on the main branch, not the Feedback branch.

whalewalker commented 1 year ago

here's the story from the start, when you sign up for Octernship there's a new repo automatically created for you, what you're supposed to do was clone that repo and work on the Feedback branch, however Houseware change their Instructions now you have to work on the main branch, not the Feedback branch.

If i work from the feedback branch, which branch will i create a PR against ? is it the feedback branch?

DevYemi commented 1 year ago

You actually don't need to create a pull request.

If you work from the feedback branch instead of the main branch it's fine, i also made that mistake, just make sure you either merge your code from feedback branch to main branch or you rebase.

For the submission:

This list above was what was written by the houseware member.

Just make sure your main branch is up to date with your latest code even if you started from the feedback branch

whalewalker commented 1 year ago

Thanks, @DevYemi. Currently, my latest changes are on the main branch and I haven't made any modifications to the feedback branch. Is that alright?

DevYemi commented 1 year ago

Yeah that's perfectly fine, that was actually the workflow explained in that list in my previous reply.

@whalewalker once you have updated your readme file with how to run the project locally and give details about any other information about the project then you are fine, the project will automatically be submitted on the deadline date

whalewalker commented 1 year ago

Yeah that's perfectly fine, that was actually the workflow explained in that list in my previous reply.

@whalewalker once you have updated your readme file with how to run the project locally and give details about any other information about the project then you are fine, the project will automatically be submitted on the deadline date

However, it is not appearing in the list of pull requests from the Houseware repository

whalewalker commented 1 year ago

Or am i missing something

DevYemi commented 1 year ago

No you ain't, it actually ain't suppose to show there.

Look at #10 pull request, a houseware member corrected the person that made the pull request there.