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.

Sign up for GitHub

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

Jump to bottom

Bug: Hovercards are broken #2138

Closed
1 of 2 tasks
bdougie opened this issue Nov 14, 2023 · 5 comments · Fixed by #2152
Closed
1 of 2 tasks

Bug: Hovercards are broken #2138

bdougie opened this issue Nov 14, 2023 · 5 comments · Fixed by #2152
Assignees
@OgDev-01
Labels
🐛 bug Something isn't working core team work Work that the OpenSauced core team takes on released on @beta released

Comments

@bdougie
Copy link
Member

bdougie commented Nov 14, 2023

Describe the bug

Noticigin in this PR's deploy preview they work, but they currently do not work in production.

#1980

image

Steps to reproduce

What the add does today.

image

Browsers

No response

Additional context (Is this in dev or production?)

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct

Contributing Docs

  • I agree to follow this project's Contribution Docs
@bdougie bdougie added 🐛 bug Something isn't working 👀 needs triage labels Nov 14, 2023
@github-actions GitHub Actions
Copy link
Contributor

Thanks for the issue, our team will look into it as soon as possible! If you would like to work on this issue, please wait for us to decide if it's ready. The issue will be ready to work on once we remove the "needs triage" label.

To claim an issue that does not have the "needs triage" label, please leave a comment that says ".take". If you have any questions, please reach out to us on Discord or follow up on the issue itself.

For full info on how to contribute, please check out our contributors guide.

@bdougie
Copy link
Member Author

bdougie commented Nov 14, 2023 •
edited
Loading

Found the bad commit through a bisect. FYI @nickytonline e1dcd55

Looks like that commit a merge from beta, but I'm not sure what is breaking the hover. Need to step away to work on some things before heading to a basketball game.

@bdougie bdougie added 👀 needs triage core team work Work that the OpenSauced core team takes on and removed 👀 needs triage labels Nov 14, 2023
@OgDev-01
Copy link
Contributor

OgDev-01 commented Nov 15, 2023 •
edited
Loading

I'll pick i up from here if you don't mind @bdougie, Was about creating an issue for this as it affects all areas the hovercard was used

@OgDev-01 OgDev-01 self-assigned this Nov 15, 2023
@OgDev-01 OgDev-01 mentioned this issue Nov 15, 2023
Merged
19 tasks
@open-sauced Open Sauced
Copy link
Contributor

open-sauced bot commented Nov 15, 2023

🎉 This issue has been resolved in version 1.76.0-beta.12 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@open-sauced Open Sauced
Copy link
Contributor

open-sauced bot commented Nov 16, 2023

🎉 This issue has been resolved in version 1.76.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@open-sauced open-sauced bot added the released label Nov 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@OgDev-01 OgDev-01

Labels
🐛 bug Something isn't working core team work Work that the OpenSauced core team takes on released on @beta released
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: hover card now works as expected in all areas open-sauced/app
2 participants
@bdougie @OgDev-01