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

Add multi link support #45

Conversation

maxime-rainville
Copy link

@maxime-rainville maxime-rainville commented May 25, 2022

This PRs add support for multi link field via an HasMany relation to Links.

It also adds some extra features like:

  • Link Types can now have icons
  • Link titles can default to picking up some bit of metadata on their target (e.g: Linking to a CMS page can default to the page title)

Parent issue

import PropTypes from 'prop-types';
import classnames from 'classnames';

const LinkBox = ({ className, children }) => (
Copy link
Author

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.

@maxime-rainville maxime-rainville marked this pull request as ready for review July 11, 2022 12:42
@maxime-rainville maxime-rainville changed the title WIP add multi link support Add multi link support Jul 11, 2022
Copy link
Collaborator

@mfendeksilverstripe mfendeksilverstripe left a comment

Choose a reason for hiding this comment

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

Tested this on vanilla install and it's working well. Really happy to see this addition 👍

Screen Shot 2022-10-28 at 11 04 02 AM

I'll leave this PR open as it looks like the implementation is not yet finished.

@edwilde
Copy link
Contributor

edwilde commented Apr 19, 2023

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.

@maxime-rainville maxime-rainville force-pushed the pull/master/multilink branch 2 times, most recently from 1e524dd to dd4cf68 Compare July 11, 2023 10:04
@maxime-rainville
Copy link
Author

maxime-rainville commented Jul 11, 2023

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

@maxime-rainville
Copy link
Author

This PR is probably redundant now.

The AnyField module does everything this thing does in a more elegant fashion.

#76 add a few extra hooks to better integrate with AnyField.

@GuySartorelli
Copy link
Member

Rebased and resolved merge conflict as part of additional work that's happening separately.

@GuySartorelli
Copy link
Member

Closing in favor of #120

@GuySartorelli GuySartorelli deleted the pull/master/multilink branch November 28, 2023 03:55
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.

4 participants