You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Overall solid work. As discussed on our 1:1s you prefer a centralized place for feedback, so let's use an issue for this assignment and take it from here.
In the README file I recommend removing the Deployed Application link if it's not applicable. Let's probably leave a comment there on the deployment status.
Let's try to have the GIF/Vid added to the README next time, here are some tutorials you might find useful - the demo is great though -:
Try to make the commit titles more action oriented - What did you do? Infinitive tense works great! You don't need to add next steps. These can be commented. An example, instead of "modal is up", we can do "[HTML][JS] Implement modal design and functionality".
Also, adding visual elements to commits, is highly recommended but optional for now - they should describe what your change is doing.
Overall solid work. As discussed on our 1:1s you prefer a centralized place for feedback, so let's use an issue for this assignment and take it from here.
The text was updated successfully, but these errors were encountered: