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

feat: Add PWA support #4

Open
mithi opened this issue Aug 31, 2020 · 9 comments
Open

feat: Add PWA support #4

mithi opened this issue Aug 31, 2020 · 9 comments

Comments

@mithi
Copy link
Owner

mithi commented Aug 31, 2020

Screen Shot 2020-08-31 at 12 48 26 PM

@mithi mithi added first-timers-only good first issue Good for newcomers up for grabs help wanted Extra attention is needed labels Aug 31, 2020
@thesanjeevsharma
Copy link

Hello @mithi, I would like to take up this issue.

@mithi
Copy link
Owner Author

mithi commented Aug 31, 2020

Hello @mithi, I would like to take up this issue.

Sure! Thank you so much! 😃

@mithi
Copy link
Owner Author

mithi commented Aug 31, 2020 via email

@thesanjeevsharma
Copy link

thesanjeevsharma commented Aug 31, 2020

In package.json, you've mentioned @mithi/bare-minimum-2d as a dependency. But It's not working. I think this is not meant to be a scoped package. You should just write bare-minimum-2d.

Is there a reason for it?

@mithi
Copy link
Owner Author

mithi commented Aug 31, 2020

In package.json, you've mentioned @mithi/bare-minimum-2d as a dependency. But It's not working. I think this is not meant to be a scoped package. You should just write bare-minimum-2d.

Is there a reason for it?

It's working in my end. Could you take a screenshot?
Yes, you can also use plain bare-minimum-2d

The reason for this is that I published this package both in the npm registry as well as the github registry.
The github registry requires that this package should be scoped

See:

Both should work (as it does on my end), but sure, if it's not working for you, you can try changing it :)
@thesanjeevsharma

@thesanjeevsharma
Copy link

Screenshot (92)

I am having trouble installing bare-minimum-3d now. It's not available on npm, so I installed it from Github. Installation is complete but if you see under node_modules; 3d is empty.

How can I solve this? Are you planning to publish it soon?

@mithi
Copy link
Owner Author

mithi commented Aug 31, 2020 via email

@mithi
Copy link
Owner Author

mithi commented Aug 31, 2020 via email

@mithi
Copy link
Owner Author

mithi commented Sep 1, 2020

@thesanjeevsharma

I have bumped the version @mithi/bare-minimum-3d to 0.1.4 and is now published at the npm registry
see https://www.npmjs.com/package/@mithi/bare-minimum-3d

If you're going to use bare-minimum-2d as opposed to @mithi/bare-minimum-2d don't forget to update App.tsx

import BareMinimum2d from "@mithi/bare-minimum-2d"

and package.json
"@mithi/bare-minimum-2d": "^0.2.0",

Thanks! 😄

Screen Shot 2020-09-01 at 8 17 32 PM

Screen Shot 2020-09-01 at 8 17 51 PM

Screen Shot 2020-09-01 at 8 18 32 PM

@mithi mithi removed the help wanted Extra attention is needed label Sep 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants