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

Allow linking between entries #68

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

marvin-roesch
Copy link
Contributor

This introduces "links" between entries, which essentially are a codified version of common relationships between entries. Right now, the following relations are implemented:

  • Refers to
  • Referred to by
  • Answers
  • Answered by
  • Corrects
  • Corrected by
  • Contradicts

As the names suggest, all these relations come in reciprocal pairs, except for contradiction, which is its own inverse.

The editing experience looks like this and as you can see it allows for specifying that a link is reciprocal, which automatically generates the corresponding link in another entry:
Link editing
The above links will be displayed as follows:
Link display
Links are fully versioned, so they are subject to the usual review process.

This pull request also contains a simple Docker Compose setup for local testing without having to provision the Vagrant VM.

@argent-codes
Copy link

I assume the Entry Links section expands on-demand?

Can you also upload a screenshot of how an existing entry would look like after it has been updated with these? You should be able to use any combination of my Renarin & Voidbinding questions:

@marvin-roesch
Copy link
Contributor Author

The Entry Links section works like the URL sources one, so new links can be added on demand.

I've added a link between two of your WoBs, which would look like this in the context of other WoBs:
Argent WoB 1
Argent WoB 2

@m-strzelczyk
Copy link
Member

@PaleoCrafter sorry I'm digging this up after so long, but if you would rebase this PR I think we can merge it finally... Sorry I left it hanging for so long :<

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.

3 participants