-
Notifications
You must be signed in to change notification settings - Fork 320
Project Labyrinth - Johanna Leonsson, Hannah Ek, Sammy Olsson #210
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
Open
Nahnahke
wants to merge
52
commits into
Technigo:master
Choose a base branch
from
Nahnahke:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
52 commits
Select commit
Hold shift + click to select a range
7db8413
updated readme
Nahnahke aec99ad
Trying first commmit
sammyolsson 05e695a
Trying first commmit again
sammyolsson d1da1ba
reducers and components
sammyolsson eb7e07a
Boilerplate
sammyolsson a89d2b1
fetch complete
Nahnahke 36dcf0e
updated
Nahnahke c31cb1a
added an unconnected input form and a button in userNameInput
Nahnahke 4169535
added usernameinput to main
Nahnahke 8303d3f
got description and coordinates up
Nahnahke f85b115
up- and running
Nahnahke 9cfd70c
finished UserNameInput
Nahnahke 8bc9ded
update
Nahnahke dbd0666
Added Lottie and much other cool things, need to get background image…
sammyolsson 14b5c56
solved the background images
Nahnahke 8b11336
started basic styling
Nahnahke 64ba6fb
changed margins
Nahnahke a50bbc3
changed font size
Nahnahke fcde945
added unique username
Nahnahke db61e69
Did some major styling
sammyolsson 81a3b8a
updated styling for mobile, added buttons
JohLeo 80a3488
responsivness for desktop
JohLeo a4c896f
removed border on illustrations lolz
JohLeo 998b769
some opacity changes
JohLeo c73c187
solved centering of GameBoardWrapper
Nahnahke 40c802b
added favicon & metatags
Nahnahke c7488a5
centered Lottie
Nahnahke 0bc3796
test
Nahnahke ba0c769
test
Nahnahke 8ab8fe4
test
Nahnahke 9275064
test
Nahnahke cdc34e3
test
Nahnahke ebfb09c
test
Nahnahke afd3e6e
test
Nahnahke 26172d0
test
Nahnahke f11adf9
test
Nahnahke 49d7c56
test
Nahnahke f6ba951
test 100
Nahnahke 90e272a
test
Nahnahke cf43d40
test
Nahnahke 30a80d5
test
Nahnahke 6589a3d
test
Nahnahke b28ebaa
test
Nahnahke 7c35e00
test
Nahnahke 13d6a8d
test
Nahnahke 5fefa00
test
Nahnahke 8e9c76f
test
Nahnahke de85501
added comments
Nahnahke ea96b11
added keys
Nahnahke 33ed4b1
test
Nahnahke 0e12920
updated readme
Nahnahke fa44baa
corrected spelling error
Nahnahke File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,13 +1,31 @@ | ||
| # Project Labyrinth | ||
| Project Labyrinth - https://project-labyrinth-jhs.netlify.app/ | ||
|
|
||
| Replace this readme with your own information about your project. | ||
| 🏁 Goals: | ||
|
|
||
| Start by briefly describing the assignment in a sentence or two. Keep it short and to the point. | ||
| In this week's project, we were tasked with building a project which allows a user to navigate a maze, in the form of a text-based adventure, provided by a backend in React and Redux and we have successfully completed what we set out to do. | ||
|
|
||
| ## The problem | ||
| The goal for this week was to use the API specified to build a frontend that gives the user control over what to do next. We focused on using thunks and redux to build the communication with the backend. | ||
|
|
||
| Describe how you approached to problem, and what tools and techniques you used to solve it. How did you plan? What technologies did you use? If you had more time, what would be next? | ||
| Overall, we found this project to be a great opportunity to practice the use of thunks, bring together different parts of React and working with fetching and posting to an API. We are very pleased with the outcome and looking forward to using the skills and knowledge we gained from this project in future endeavors. | ||
|
|
||
| ## View it live | ||
| Must-have requirements: | ||
|
|
||
| Every project should be deployed somewhere. Be sure to include the link to the deployed project so that the viewer can click around and see what it's all about. | ||
| ✅ Build a responsive page. | ||
|
|
||
| ✅ Use Redux to store the current state of the game. | ||
|
|
||
| ✅ Focus on the UX design and handle response delay. | ||
|
|
||
| ✅ Use thunks to wrap the API calls. | ||
|
|
||
| 🚧 The problem: | ||
|
|
||
| We started by going through the design and sketching the different components out along with the design, deciding on how to structure the app. We used a mob-programming approach, starting off by fetching from the API using thunks. Then we moved on to build the base of the code structure and building the LoadingIndicator.js using a Lottie. Then we put time and effort into styling the page and making sure everything works as it should. | ||
|
|
||
| If we had more time, we would have added arrows that point the player into the direction displayed. We would also have added a background song where the player is able to toggle the sound on and off. | ||
|
|
||
| 💻 Tech stack: | ||
|
|
||
| React Redux CSS HTML Figma | ||
|
|
||
| View it live https://project-labyrinth-jhs.netlify.app/ | ||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good looking readme. Clear goals and descriptions of how you solved the task.