-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
chore: modify the darktheme in the add blog and fixed the eslint error #374
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Hey @Ameerjafar! Thanks for sticking to the guidelines! High five! 🙌🏻 |
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.
LGTM, couple of changes
@@ -1,6 +1,6 @@ | |||
import { HTTP_STATUS, RESPONSE_MESSAGES } from '../utils/constants.js'; | |||
|
|||
const errorMiddleware = (err, req, res, next) => { | |||
const errorMiddleware = (err, req, res) => { | |||
console.error(err.stack); |
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.
We can't do this, because the default error middleware which the express figures out is supposed to have 4 params, We need to compress just on this line or you can call next() which anyways doesn't make much sense
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.
done sir
@@ -130,7 +130,7 @@ function AddBlog() { | |||
}); | |||
|
|||
return (await postPromise).data; | |||
} catch (error: any) { | |||
} catch (error: unknown) { | |||
if (isAxiosError(error)) { |
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.
Can you figure out if you can know it ...
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.
LGTM, thank you :)
Summary
modify the dark theme in the add blog page. and cleared the eslint error in both frontend and backend code.
Description
I have changed the dark theme in the add blog component at the same time i have changed the color of login and signup button to white and cleared the eslint error in both frontend and backend code.
Images
signin page



signup page
add blog page
Prerequisites