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

V2: added snapshot versions + config #150

Draft
wants to merge 108 commits into
base: master
Choose a base branch
from
Draft

V2: added snapshot versions + config #150

wants to merge 108 commits into from

Conversation

shahednasser
Copy link
Member

No description provided.

@shahednasser shahednasser changed the title added snapshot versions + config V2: added snapshot versions + config Mar 18, 2024
kodiakhq bot pushed a commit to medusajs/medusa that referenced this pull request Mar 25, 2024
## What

Added a `--v2` option to the `create-medusa-app` command that clones the [feat/v2](medusajs/medusa-starter-default#150) branch of the starter default and makes minor changes to the setup process.

## Why

This option is supposed to make it easier to test out internally a Medusa setup with all the commerce modules. The starter's branch installs and configures the modules + the V2 feature flag.

## Testing

To test it out, run the snapshot with the `--v2` option.

## Notes

- I couldn't install the new admin dashboard in the starter branch as it's not on NPM yet (at the time of writing this), so at the moment, this opens the current admin dashboard instead.
- When trying to create an invite using the CLI tool I get the error `Unable to resolve inviteService`. Not sure if we should also make changes to the CLI tool to allow creating an invite when V2 is enabled (I know now invites are creating within the User Module, so maybe when the V2 feature flag is enabled that should be resolved instead), but for now I just disabled running the create invite command as this is mainly for internal testing. Let me know what the suggested approach is here.
shahednasser and others added 30 commits May 27, 2024 14:17
Co-authored-by: shahednasser <shahednasser@users.noreply.github.com>
* test actions

* fix script

* revert README

* update yarn.lock

* fix install command for npm

* fixes

* try fix

* fix permissions

* remove resolutions

* update test names

* update loack file

* remove cli as dependency

---------

Co-authored-by: adrien2p <adrien.deperetti@gmail.com>
Co-authored-by: GitHub <github-actions[bot]@users.noreply.github.com>
feat: Add default store sales channel
Co-authored-by: GitHub <github-actions[bot]@users.noreply.github.com>
chore(feat/v2): updated preview dependencies [automated]
Co-authored-by: GitHub <github-actions[bot]@users.noreply.github.com>
Co-authored-by: GitHub <github-actions[bot]@users.noreply.github.com>
Co-authored-by: GitHub <github-actions[bot]@users.noreply.github.com>
Co-authored-by: GitHub <github-actions[bot]@users.noreply.github.com>
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.

None yet

5 participants