Skip to content
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] update market binaries #107

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

Conversation

PatMulligan
Copy link
Contributor

@PatMulligan PatMulligan commented Nov 5, 2024

}
}

function productCompare(a, b, sortBy, sortOrder) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

why was this added?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I didn't modify anything.

looks like it was done in lnbits/nostr-market-app#11

@PatMulligan
Copy link
Contributor Author

hey, been traveling a bit so just getting back to check this out.

Still a bit of a newb when it comes to properly handling PR's, as you can see... I see I should have tested it out in the context of running in lnbits and not stand-alone for the paths!

Let me know if there is anything I can do to help finish up the PR.

@PatMulligan
Copy link
Contributor Author

PatMulligan commented Nov 30, 2024

okay so this has been a learning experience 😅 when building nostr-market-app it's necessary to change the following in quasar.config.js before building

      publicPath: "/nostrmarket/static/market",
      // publicPath: "/",
    framework: {
      config: { staticPath: "/nostrmarket/static/market/" },
      // config: { staticPath: "" },

so that it builds correctly in the context of running inside lnbits.

At first I just went in and manually added the paths, but then I don't know if the build names the files, e.g., index.70852868.js specifically as a hash or something and didn't want to screw up things down the line just in case, so I just changed the above, rebuilt, replaced, etc.

Tested it out a bit, gonna test some more, but otherwise should be pretty good to go!

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.

Wrong value in checkout when adding more than one unit of a product
2 participants