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

update repo #115

Merged
merged 0 commits into from
Apr 17, 2024
Merged

update repo #115

merged 0 commits into from
Apr 17, 2024

Conversation

Vadman97
Copy link
Member

No description provided.

@Vadman97 Vadman97 force-pushed the vadim/rrweb-update-repo branch 5 times, most recently from 1bd08cb to c6e4b94 Compare January 30, 2024 19:01
@Vadman97 Vadman97 force-pushed the vadim/rrweb-update-repo branch 2 times, most recently from 0d5cd39 to 7384bca Compare March 5, 2024 19:10
@Vadman97 Vadman97 marked this pull request as ready for review March 5, 2024 19:38
Vadman97 added a commit to highlight/highlight that referenced this pull request Mar 12, 2024
## Summary

highlight/rrweb#115

hoping this will fix some [scroll
issues](highlight/rrweb@e607e83)
reported by users recently, as well as some canvas recording issues
fixed in rrweb

renames our rrweb forks back to the original names to make it easier to
rebase in the future.
we will still be using our local fork via the `"rrweb": "workspace:*"`
dependencies in the `package.json`s


## How did you test this change?

in progress testing via reflame and local deploy

![image](https://github.com/highlight/highlight/assets/1351531/fe8f1e95-e3ae-465f-aad3-586d70c6498c)

![image](https://github.com/highlight/highlight/assets/1351531/6ed3c8e0-7ff4-4675-9f94-300e856997e3)

![image](https://github.com/highlight/highlight/assets/1351531/ce2e7fe0-97d1-49f7-8ae3-42e022ac32d9)


## Are there any deployment considerations?

changeset

## Does this work require review from our design team?

no
@Vadman97 Vadman97 force-pushed the vadim/rrweb-update-repo branch from 7384bca to 9817409 Compare April 16, 2024 19:50
@Vadman97 Vadman97 force-pushed the vadim/rrweb-update-repo branch from 9817409 to 7535ca1 Compare April 17, 2024 01:11
@Vadman97 Vadman97 merged commit 7535ca1 into master Apr 17, 2024
4 of 9 checks passed
Vadman97 added a commit to highlight/highlight that referenced this pull request Apr 17, 2024
## Summary

Fix `.tagName` null access issue reported on
[discord](https://discord.com/channels/1026884757667188757/1229747689353252875)
via rrweb-io/rrweb#1445

highlight/rrweb#115

Closes HIG-4504

## How did you test this change?

reflame

## Are there any deployment considerations?

changeset

## Does this work require review from our design team?

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

1 participant