Skip to content

fix(app): display prerequisites on edit screen - #364

Draft
TheGIftedExplorer wants to merge 4 commits into
bluelearn-org:mainfrom
TheGIftedExplorer:fix/display-prerequisites-on-edit
Draft

fix(app): display prerequisites on edit screen#364
TheGIftedExplorer wants to merge 4 commits into
bluelearn-org:mainfrom
TheGIftedExplorer:fix/display-prerequisites-on-edit

Conversation

@TheGIftedExplorer

@TheGIftedExplorer TheGIftedExplorer commented Aug 23, 2026

Copy link
Copy Markdown
Contributor

Summary

Made the prerequisite guides(both exising and to-do) visible on the Edit Guide screen without allowing them to be modified. Added a prop showPrerequisiteFields to separate visibility from editability

Type of change

  • Bug fix
  • Feature
  • Refactor (no behavior change)
  • Documentation
  • Build / tooling / CI
  • Other: _________

Verification

  • pnpm -r typecheck passes
  • pnpm -r build passes
  • Manually verified in a browser (for UI / behavior changes)
  • Followed the app layout conventions
  • No new dependencies, or new dependencies are justified and AGPL-compatible
  • Change does not violate any non-negotiable principle
  • Documentation only, no changes to the codebase, so no tests required

Closes #363 #321

Signed-off-by: TheGiftedExplorer <vedarthasai90@gmail.com>

@andyrodrigues30 andyrodrigues30 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@TheGIftedExplorer - Both prerequisite guides and todo prerequisite guides fields should be editable, right now the fields are disabled.

Image

@TheGIftedExplorer
TheGIftedExplorer marked this pull request as draft August 26, 2026 03:21
@andyrodrigues30

Copy link
Copy Markdown
Collaborator

@TheGIftedExplorer - is there an update on this PR?

@TheGIftedExplorer

Copy link
Copy Markdown
Contributor Author

@andyrodrigues30 Worked with this graph API I came across
Added a GET endpoint to fetch prerequisite graph edges
Added the corresponding frontend API helpers
Updated the guide edit flow to sync prerequisite changes through the graph API instead of syncDraftTagsAndEdges().
Adding new prerequisites works correctly in an updated draft but while testing removal of an existing prerequisite, I found that the frontend sync logic attempts to remove the edge, but the database RLS only allows moderators/admins to update guide_edges.

There may be an intended workflow I'm missing. I am leaving this issue for someone with good understanding of the backend

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.

[Bug] Prerequisites fields missing in edit form [Feature] Allow users to edit todos and normal prerequisites under guides

2 participants