Skip to content

Comments

Add wiki pages #5904

Open
lukasmueller wants to merge 32 commits intomasterfrom
topic/wiki_pages
Open

Add wiki pages #5904
lukasmueller wants to merge 32 commits intomasterfrom
topic/wiki_pages

Conversation

@lukasmueller
Copy link
Member

@lukasmueller lukasmueller commented Feb 12, 2026

Description

Adds a wiki page system at /wiki.
Uses new tables, sgn_people.sp_wiki and sgn_people.sp_wiki_content installed by a dbpatch.
Needs modified cxgn-corelibs, CXGN::People::Schema::Result::SpWiki and CXGN::People::Schema::Result::SpWikiContent. See pull request in cxgn-corelibs. Also needs a new Perl module, Text::MultiMarkdown, which has been added through apt to the Dockerfile.

Checklist

  • Refactoring only
  • Documentation only
  • Fixture update only
  • Bug fix
    • The relevant issue has been closed.
    • Further work is required.
  • New feature
    • Relevant tests have been created and run.
    • Data was added to the fixture
      • Data was added via a patch in /t/data/fixture/patches/.
    • User-Facing Change
      • The user manual in /docs has been updated.
    • Any new Perl has been documented using perldoc.
    • Any new JavaScript has been documented using JSDoc.
    • Any new legacy JavaScript has been moved from /js to /js/source/legacy.

@bm743
Copy link
Contributor

bm743 commented Feb 16, 2026

I am getting "An error occurred" messages when creating a new wiki page, and clicking the edit button

@lukasmueller
Copy link
Member Author

lukasmueller commented Feb 17, 2026

It seems that Text::MultiMarkdown was missing from the Docker. I added it using apt (libtext-multimarkdown-perl) in the Dockerfile.

You will need to update the docker to run it correctly.

Copy link
Contributor

@ryan-preble ryan-preble left a comment

Choose a reason for hiding this comment

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

I got an error when trying to access the wiki page:

Image

I imagine the dbpatch needs a GRANT statement

@lukasmueller
Copy link
Member Author

lukasmueller commented Feb 17, 2026 via email

@lukasmueller
Copy link
Member Author

Added the grant statements to the dbpatch. You probably have to manually remove the tables in the database and force run the dbpatch again

@ryan-preble
Copy link
Contributor

Ran into a few little errors.

When loading up the wiki page for the first time, it prompted me to create the wikihome page. I did that, saved it, and edited it successfully.

When I clicked "new" I was prompted to name the new page, and the editor autofilled with the wikihome markdown text. When I tried to edit this text into my new page, I was given some strange errors about ownership over the wiki page (I was logged in as Jane Doe). I then tried to go in and repeat the procedure to create the error, but I got this instead:
image

So I went to "all pages", deleted that wiki entry, and then tried again, but was then given this:
image

Seems there is something going wrong with storing the user ID in association with the wiki page.

@lukasmueller
Copy link
Member Author

I fixed the issue with the content from the previous wiki page. I have not yet been able to reproduce the ownership issue... Have you logged in with two different users or just one?

@ryan-preble
Copy link
Contributor

I fixed the issue with the content from the previous wiki page. I have not yet been able to reproduce the ownership issue... Have you logged in with two different users or just one?

Only logged in with one user

@lukasmueller
Copy link
Member Author

I see the owner issues now... thanks for catching that

@lukasmueller
Copy link
Member Author

I think I addressed all the issues. Please check again.

@ryan-preble ryan-preble self-requested a review February 19, 2026 14:23
@lukasmueller
Copy link
Member Author

Sometimes initial page does not exist error...

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