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

refactor: rename components package to smart camera web #140

Merged
merged 8 commits into from
Nov 28, 2023

Conversation

tamssokari
Copy link
Contributor

No description provided.

@tamssokari tamssokari self-assigned this Nov 26, 2023
Copy link

@@ -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",
Copy link
Contributor

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.

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 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.

Copy link
Contributor Author

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.

tams sokari and others added 2 commits November 26, 2023 16:03
* 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)
tamssokari and others added 2 commits November 26, 2023 16:46
* 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
Copy link
Contributor

@mldangelo mldangelo left a comment

Choose a reason for hiding this comment

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

Thank you!

@tamssokari tamssokari merged commit e29de42 into main Nov 28, 2023
7 checks passed
@tamssokari tamssokari deleted the packages/rename-components-to-smart-camera-web branch November 28, 2023 13:34
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