-
Notifications
You must be signed in to change notification settings - Fork 15
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
Add multi link support #45
Add multi link support #45
Conversation
import PropTypes from 'prop-types'; | ||
import classnames from 'classnames'; | ||
|
||
const LinkBox = ({ className, children }) => ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add some comment to describe what this does.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey, is there a timeline to get this merged? I am using linkfield in a project which has a few of these and the experience is pretty clunky, I am a little worried the client might request we go back to linkable because of it. |
1e524dd
to
dd4cf68
Compare
I've got innovation week in a fortnight. I'll aim to get this across the line then. FYI I've rebased the thing to work with CMS5 |
b2749eb
to
f984376
Compare
Rebased and resolved merge conflict as part of additional work that's happening separately. |
Closing in favor of #120 |
This PRs add support for multi link field via an HasMany relation to Links.
It also adds some extra features like:
Parent issue