-
Notifications
You must be signed in to change notification settings - Fork 2
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
refactor: rename components package to smart camera web #140
refactor: rename components package to smart camera web #140
Conversation
This branch has been deployed to s3 / cloudfront. ✅ Preview URL for Smart Camera Web: |
packages/embed/package.json
Outdated
@@ -27,7 +27,7 @@ | |||
}, | |||
"homepage": "https://github.com/smileidentity/hosted-web-integration#readme", | |||
"dependencies": { | |||
"@smile_identity/smart-camera-web": "file:../components", | |||
"@smile_identity/smart-camera-web": "^1.0.2", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are you sure about this change? It's going to make publishing a lot more complicated.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have tested and verified that it uses the local package as we have this set up as a workspace.
I see the concerns around possible version collisions, will test that shortly.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tested, and returning to file path.
while this is the right direction for workspaces, we need to avoid situations where updates
to workspace packages do not reflect in dependent packages.
for now, local paths are the better option.
* refactor: move combobox component to components package * ci: deploy preview on PRs to all branches * deps: add `@smileid/components` as dependency of `@smileid/embed` * refactor: use local paths for workspace dependencies * refactor: extract `end-user-consent` component to `components` package (#142)
* refactor: move combobox component to components package * ci: deploy preview on PRs to all branches * deps: add `@smileid/components` as dependency of `@smileid/embed` * refactor: extract `end-user-consent` component to `components` package * refactor: extract totp-consent to components package * refactor: use `<totp-consent>` as the markup * refactor: rename `totp-consent` in tests
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
No description provided.