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

[WIP] - Arranger Dev Documentation #892

Draft
wants to merge 5 commits into
base: develop
Choose a base branch
from
Draft

Conversation

MitchellShiell
Copy link
Contributor

@MitchellShiell MitchellShiell commented Sep 23, 2024

Summary

Updated developer documentation for Arranger

Description

  • Rendered via submodules linked in the bridge repository and deployed here
  • This combines the usage sections of the product documentation on Overture.bio with new setup and Overview pages for each microservice (this branch/PR focuses on Arranger related documentation)
  • Setup uses the Conductor service for spinning up complementary services

To Do

With these items are completed I will finish the initial docs and follow-up for a more thorough review

Running the site locally (If you choose so)

Documented on the documentation site (so meta) here

Copy link
Member

@justincorrigible justincorrigible left a comment

Choose a reason for hiding this comment

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

that's my ¢2

└── scripts/
```

- **`docker/`**: Contains Docker configurations for various components.
Copy link
Member

@justincorrigible justincorrigible Sep 26, 2024

Choose a reason for hiding this comment

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

Suggested change
- **`docker/`**: Contains Docker configurations for various components.
- **`docker/`**: Contains miscellaneous configuration files used for building Docker images of Arranger Server, and to support running a local developer environment.

- **`docker/`**: Contains Docker configurations for various components.
- **`integration-tests/`**: Houses integration tests for system-wide functionality.
- **`modules`/**: Core Arranger modules:
- **`admin-ui/`**: Administration interface for managing Arranger configurations.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- **`admin-ui/`**: Administration interface for managing Arranger configurations.
- **`admin-ui/`**: (Inactive) Administration interface for generating and managing Arranger configuration files.

- **`modules`/**: Core Arranger modules:
- **`admin-ui/`**: Administration interface for managing Arranger configurations.
- **`components/`**: React components for building search UIs.
- **`server/`**: Arranger Server implementation (GraphQL API).
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- **`server/`**: Arranger Server implementation (GraphQL API).
- **`server/`**: the "Arranger" server itself, a GraphQL service that facilitates usage of Lucene-based search engines (e.g. Elasticsearch).

- **`integration-tests/`**: Houses integration tests for system-wide functionality.
- **`modules`/**: Core Arranger modules:
- **`admin-ui/`**: Administration interface for managing Arranger configurations.
- **`components/`**: React components for building search UIs.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- **`components/`**: React components for building search UIs.
- **`components/`**: React components to streamline integration of search portals with an Arranger server.

```

- **`docker/`**: Contains Docker configurations for various components.
- **`integration-tests/`**: Houses integration tests for system-wide functionality.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- **`integration-tests/`**: Houses integration tests for system-wide functionality.
- **`docs/`**: Markdown files that contain instructions on how to use Arranger and its capabilities, contribution guidelines, etc.
- **`integration-tests/`**: Houses integration tests for system-wide functionality.

│ ├── server/
│ ├── test/
│ └── ui/
├── integration-tests/
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
├── integration-tests/
├── docs/
├── integration-tests/

@MitchellShiell MitchellShiell changed the title [WIP] - Arranger Dev Docs Update [WIP] - Arranger Dev Documentation Sep 27, 2024
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