Skip to content

Conversation

@Harshita0008
Copy link
Owner

No description provided.

@vercel
Copy link

vercel bot commented Mar 27, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
merchant-form ❌ Failed (Inspect) Mar 27, 2024 9:05am

@shawnmuggle
Copy link
Contributor

  1. You PR contains the change of Yahya's change. Only send out your code changes.

merge xxx commits into Harshita0008:main from MugglePay:feature
=> merge xxx commits into MugglePay:main from MugglePay:feature

  1. Here are the feedback I sent earlier.
    For your style check, would you use space instead of tabs? Don't use tabs in space.
    It's making a lot of style changes that are unrelated.

If you want to make style changes, make a PR only for style change; and then another one for your changes.

When you create a PR, please describe your changes and your expected goal for the CL.

}
// Default options are marked with *
const response = await fetch('https://api.muggle.link/api/products', {
const response = await fetch('http://127.0.0.1:3001/api/products', {
Copy link
Contributor

Choose a reason for hiding this comment

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

?

Copy link
Contributor

@shawnmuggle shawnmuggle left a comment

Choose a reason for hiding this comment

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

You are using localhost.
You can add a config file (config or .env)

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