Skip to content

doc: add an entrypoint for the internal API documentation#1525

Draft
tahini wants to merge 1 commit into
chairemobilite:mainfrom
tahini:updateJsdoc
Draft

doc: add an entrypoint for the internal API documentation#1525
tahini wants to merge 1 commit into
chairemobilite:mainfrom
tahini:updateJsdoc

Conversation

@tahini
Copy link
Copy Markdown
Contributor

@tahini tahini commented Apr 23, 2026

Add a docs/surveyAdminDoc.md file, that will be the entrypoint readme of the Evolution API documentation. This page is meant to be read in the site generated by the typedoc package and targets the survey designers, with instructions on how to implement a new survey. It is not a developer documentation, nor a user documentation.

Move the README.md from packages/evolution-generator to the docs, under the generator.md name, so it can be added and linked to by the typedoc documentation.

Add a `docs/surveyAdminDoc.md` file, that will be the entrypoint readme
of the Evolution API documentation. This page is meant to be read in the
site generated by the `typedoc` package and targets the survey
designers, with instructions on how to implement a new survey. It is not
a developer documentation, nor a user documentation.

Move the README.md from `packages/evolution-generator` to the docs,
under the `generator.md` name, so it can be added and linked to by the
typedoc documentation.
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 23, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 9868a504-4e40-450a-9000-29cca019981d

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@tahini
Copy link
Copy Markdown
Contributor Author

tahini commented Apr 23, 2026

Draft for now, questions to consider:

  1. @samuel-duhaime if we move the generator README "here", in the docs, is it ok?
  2. Is it ok to put this in the docs folder or should we differentiate the "Evolution" documentation (with concepts, survey methodology, nomenclature, info on audits, info aimed at developers) from the "survey designer" doc, whose audience are people wanting to create a new survey with Evolution? And add a different folder for this?
  3. The markdown files modified here contain links that would be broken if clicked on from the Evolution github repo, but would work only in the context of the typedoc documentation. I don't know how we can reconcile that. TBD, typedoc may have a solution.
  4. Shall we provide code example, or place holders in the markdown that can be filled automatically by code that is certain to compile (for example for the QuestionnaireFactory code, we could use the demo_survey files with some tags to highlight the parts to put here). We're lazy, we don't want to maintain doc. But that can come later, a first version could contain simply no code.

@tahini
Copy link
Copy Markdown
Contributor Author

tahini commented Apr 23, 2026

Aussi @samuel-duhaime est-ce que rouler le générateur dépend de la présence du fichier README.md pour vrai?

@tahini
Copy link
Copy Markdown
Contributor Author

tahini commented Apr 24, 2026

This would fix #1347

Copy link
Copy Markdown
Collaborator

@samuel-duhaime samuel-duhaime left a comment

Choose a reason for hiding this comment

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

@tahini Yes, it's okay, just make sure to change the readme path from pyproject.toml.

NB: you could probably also change my name from the author list. I'm not the only one now.

image

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.

2 participants