-
Notifications
You must be signed in to change notification settings - Fork 0
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
Review of assessment #1
Comments
I couldn't access your application, it had an app loader that wont stop.
|
Ok thanks, i will fix the issues as soon as possible.
…On Thu, 21 Jan 2021 at 01:58, Adejo David ***@***.***> wrote:
I couldn't access your application, it had an app loader that wont stop.
- Your code looks okay but lots of repetition and no optimisation.
- Most frameworks and library help with browser compatibility, your
css had lots of compatibility check which was an overhead.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#1 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AD55GUU4N672TAVVBI2GBATS2532LANCNFSM4V7XZBSQ>
.
|
Please recheck, you can access it now https://dashboard-3470d7.netlify.app/. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Please @dreplica @markeu I would like you to review my work. I'm done with the features outlined including deployment https://dashboard-3470d7.netlify.app/. if there's any other thing I need to do, I would appreciate it if you can point that out. thanks.
James Ononiwu.
[email protected]
The text was updated successfully, but these errors were encountered: