Skip to content

new: added new codes#9

Open
Chidubem100 wants to merge 4 commits into
CodeBrosss:mainfrom
Chidubem100:main
Open

new: added new codes#9
Chidubem100 wants to merge 4 commits into
CodeBrosss:mainfrom
Chidubem100:main

Conversation

@Chidubem100
Copy link
Copy Markdown
Contributor

made some new changes to the code base. check it out

Copy link
Copy Markdown
Owner

@CodeBrosss CodeBrosss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for contributing. We're making use of cloudinary now for storing product images and admin photo.

There's a mistake in the models/paystack where you're exporting the schema.

Comment thread models/paystack.js Outdated
const Paystack = mongoose.model("Paystack", paystackSchema);
module.exports = Paystack; No newline at end of file
// const Paystack = mongoose.model("Paystack", paystackSchema);
module.exports = mongoose.modele("Paystack", paystackSchema); // shorter codes like this runs faster. No newline at end of file
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the tip

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's a typo in "mongoose.modele", please be more careful, check that everything is good before making a pull request. Thanks .

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok..will resolve it

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

did you see another pull request?

@Ikem-coded-it
Copy link
Copy Markdown
Collaborator

What's up bro, thanks for your help for real. I'm actually done with the API. Merging your code might cause some problems because i've gone past the point where the contribution fits with the rest of the code, so merging it is bound to cause bugs that will be a burden to fix because i've been busy these past few days fixing bugs. Please i can't risk more bugs, i hope you understand. But i implemented your code manually. I added your updated error handler. Thanks for your help again.

@Ikem-coded-it
Copy link
Copy Markdown
Collaborator

If you would like to work with me on my next project please chat me on whatsapp 08126334933. It's a project i wanna build with some other people and its actually something big and requires a good number of hands on deck. If you're interested please chat me up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants