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

OUT-106 Add Embed Type #59

Merged
merged 4 commits into from
Mar 26, 2024
Merged

OUT-106 Add Embed Type #59

merged 4 commits into from
Mar 26, 2024

Conversation

aatbip
Copy link
Contributor

@aatbip aatbip commented Mar 25, 2024

@aatbip aatbip self-assigned this Mar 25, 2024
Copy link

vercel bot commented Mar 25, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
client-home ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 26, 2024 8:07am

@aatbip aatbip changed the title OUT-213 Add Embed Type OUT-106 Add Embed Type Mar 25, 2024
Copy link
Collaborator

@rrojan rrojan left a comment

Choose a reason for hiding this comment

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

Please fix the lint issues causing build fail.

Most likely it is related to tweaking the prettier config in an earlier commit

@aatbip
Copy link
Contributor Author

aatbip commented Mar 25, 2024

Please fix the lint issues causing build fail.

Most likely it is related to tweaking the prettier config in an earlier commit

Linter is turned off on this branch. Some minor changes are made with Linters in #58. After the merge, linter should get restarted.

@aatbip aatbip requested a review from rrojan March 25, 2024 12:38
return {
setIframe:
(options: { src: string }) =>
({ tr, dispatch }) => {

Choose a reason for hiding this comment

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

I see formating error here @aatbip

@saugatmaharjan
Copy link

@aatbip please address lint and type warnings.

@aatbip
Copy link
Contributor Author

aatbip commented Mar 26, 2024

@aatbip please address lint and type warnings.

@saugatmaharjan This PR will get merged to #58 which has the linters setup.

@aatbip aatbip merged commit a518819 into OUT-213 Mar 26, 2024
2 of 3 checks passed
aatbip added a commit that referenced this pull request Mar 26, 2024
* fix: editor scrolls between mode changes

* fix: editor scrolls between mode changes

* default state client.givenName change to pill

* background picker z index fix

* added swr

* fix z index of select

* fix editable false in client preview

* setup prettier with eslint again

* turned off indent rule

* setup prettier again + removed semi

* changes in eslint and prettier config

* fix indentation

* fix indentation

* refac eslintrc config

* removed identation from eslintrc

* removed unused vars

* OUT-106 Add Embed Type (#59)

* added embed functionality

* added Iframe extension to client preview

* added cursor auto to readonly mode in widget

* use promise.all in client preview server component
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