Closed XanderRubio closed 1 year ago
The latest updates on your projects. Learn more about Vercel for Git βοΈ
Name | Status | Preview | Comments | Updated (UTC) |
---|---|---|---|---|
before-i-die-achievements | β Ready (Inspect) | Visit Preview | π¬ Add feedback | Sep 6, 2023 9:51pm |
This pull request successfully closes issue #48, and thank you to contributor @negar-75 for her pull request #52 and adding UI enhancement.
so now everything has been solved?
this is my first time working with Git and Git flow and that was hard experience πππππ
so now everything has been solved?
Yes, it has! Please look at the live link for the website for proof.
although I have no idea about what just happened , but that was practical experience πππ
I see!! amazingππππ
I am so ready to make another change !!!ππππ and learn more and more about Git flow
I see!! amazingππππ
Congratulations on gaining more experience with Git today than you might have thought possible when starting out. I will do my best to provide a brief description of what happened. The main issue we encountered was that your forked version and add features branch of issue-#48 hadn't been fetched to be udaped with the main working branch of the Before I Die Code repo. This was a valuable lesson for me, and we will need to have proper documentation in place to make it easier for other developers to understand and receive support on how to prevent merge conflicts from happening. If conflicts do occur, the only option is to merge the main branch into the feature branch, potentially losing the work that you want to add to the main branch and needing to revert the commit. I need to understand why this is occurring. Please let me know if you have any further questions. I will be signing off for tonight, but I would like to hear your thoughts on how we can continue to improve the UI to look more professional and sleek. Have a good night and thank you for your time and patience! I will be adding a LinkedIn recommendation later as well.
thanks @XanderRubio for your informative text and I appreciate it, Now I am going to create another merge request for auto translation
thanks @XanderRubio for your informative text and I appreciate it, Now I am going to create another merge request for auto translation
Great @negar-75 and my pleasure!
@negar-75 added code from pull request #52 to enhance the UI