-
Notifications
You must be signed in to change notification settings - Fork 1
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
Initial Documentation Content #346
Conversation
f46334b
to
42d4669
Compare
@@ -40,7 +40,6 @@ flowchart LR; | |||
GS[Graph Service] | |||
CPS[Content Publishing Service] | |||
CWS[Content Watcher Service] | |||
RS[Graph Reconnection Service] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removing reconnection service for now
@@ -0,0 +1,60 @@ | |||
/* Button Links */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Button Link support
services/account/ENVIRONMENT.md
Outdated
This application recognizes the following environment variables: | ||
|
||
| Name | Description | Range/Type | Required? | Default | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Formatted and sorted
This application recognizes the following environment variables: | ||
|
||
| Name | Description | Range/Type | Required? | Default | | ||
| ----------------------------------------- | ---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- | :------------------------------: | :-------: | :-------------------------: | | ||
| `API_PORT` | HTTP port that the application listens on | 1025 - 65535 | | 3000 | | ||
| `CACHE_KEY_PREFIX` | Prefix to use for Redis cache keys | string | | content-publishing-service: | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Formatted and sorted
This application recognizes the following environment variables: | ||
|
||
| Name | Description | Range/Type | Required? | Default | | ||
| ----------------------------------------- | --------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- | :------------------------------: | :-------: | :--------------: | --- | --- | --- | --- | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed a typo, formatted, and sorted.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!!
added some base content to open the flood gates
Minor updates and suggestions.
8fa0a3c
to
7f65730
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Haven't been through all of it but here are some suggestions.
Account Service is a service enabling easy interaction with accounts on Frequency. | ||
Accounts are be defined as an `msaId` (a 64 bit identifier) and can contain additional information such as a handle, keys, and more. | ||
|
||
- Account creation using [SIWF](https://github.com/ProjectLibertyLabs/siwf) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it also for authentication?
``` | ||
|
||
```sh | ||
cd social-web-template |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo, should be social-app-template
- From #346 comments - Fixing issues with deployment: https://github.com/ProjectLibertyLabs/gateway/actions/runs/10404141334
This is the first pile of the documentation content.
Testing
Closes Docs issues:
Closes embed issues: