-
-
Notifications
You must be signed in to change notification settings - Fork 367
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
Most likely this will be pushed to production as well #5659
Conversation
…ionType/GalleryItemRelist.vue
…ionType/GalleryItemTransfer.vue
feat: removing the extra padding above the first event on mobile
✅ Deploy Preview for koda-canary ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
✅ Deploy Preview for koda-beta ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
New dependency changes detected. Learn more about Socket for GitHub ↗︎ 👍 No new dependency issues detected in pull request Bot CommandsTo ignore an alert, reply with a comment starting with Pull request alert summary
📊 Modified Dependency Overview:
|
wait, checking |
AI-Generated Summary: This pull request involves changes in multiple files related to updating import statements for the
Overall, these updates focus on maintaining compatibility with newer versions of the |
Reviewpad Report
|
fix: Transfer doing weird thing
AI-Generated Summary: This pull request includes updates to import statements related to the '@kodadot1/minimark' package, with most imports now coming from versioned paths ('v1', 'v2' or 'common'), enhancements to various components and styles, and improved support for different versions of RMRK interactions. Notable changes include improvements to the CarouselIndex component, Addition of a substack icon in TheFooter.vue, reorganization of code inside templates for better conditional rendering, and modifications in several utility files to handle multiple RMRK versions. Additionally, there are adjustments to chains and prefixes in the 'utils/chain.ts' file, an update of the |
only this thing probably can break something? |
@vikiival said it's oke ;D |
just want to make sure, we include this also, right? #5632 and one thing, on production, it will show rmrk and bsx, is that correct? otherwise, lgtm edit: eh wait, on beta got this, is that expected? or is beta still not deployed yet? |
chore: update reviewpad.yml
yes already merged
yes
not sure what I should see on picture? yet this PR is not merged :) |
aahh I see 🙏🏻 |
will be merging this probably |
Speak if you want to tell something before last push