Skip to content

Conversation

maparent
Copy link
Collaborator

@maparent maparent commented Aug 5, 2025

This is documenting how the references to discourse node instances would look in the database.

Some more context:

  1. I think that content links are a pure content concern, and should cover (say) arbitrary hyperlinks
  2. In theory, identifying concepts occurring inside content may not go through an explicit hyperlink, and may be a matter of interpretation. As such it makes sense for it to go from content directly to concept.
  3. In the (common) case of a hyperlink to content that represents a concept, we may either materialize both or infer the occurence. I am still thinking about the complexity and performance tradeoffs here.
  4. I believe it is worth it to materialize the hyperlink anchor in the database, but again there are complexity tradeoffs.

This documentation is not meant to be a definitive answer to those questions but to have a support to discuss options.

Copy link

linear bot commented Aug 5, 2025

Copy link

supabase bot commented Aug 5, 2025

This pull request has been ignored for the connected project zytfjzqyijgagqxrzbmz because there are no changes detected in packages/database/supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@maparent maparent requested a review from mdroidian August 5, 2025 17:17
@maparent maparent changed the title Diagrams to show reference implementation. ENG 696 Diagrams to show possible implementations of reference Aug 5, 2025
@maparent maparent force-pushed the eng-696-scoping-and-documenting-options branch from 0460559 to 748fb65 Compare August 7, 2025 14:20
@maparent
Copy link
Collaborator Author

maparent commented Aug 7, 2025

did a few changes to harmonize conventions with ENG-704

@maparent maparent force-pushed the eng-696-scoping-and-documenting-options branch 2 times, most recently from 21e8558 to 6a8ee96 Compare August 11, 2025 02:15
Copy link
Contributor

Deferred for now.

@mdroidian mdroidian marked this pull request as draft August 14, 2025 19:45
@maparent maparent force-pushed the eng-696-scoping-and-documenting-options branch from 6a8ee96 to 27f80e0 Compare August 16, 2025 02:05
@maparent maparent force-pushed the eng-696-scoping-and-documenting-options branch from 27f80e0 to 914fce3 Compare August 24, 2025 17:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants