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

Docs: adding veterans use case #1748

Merged
merged 10 commits into from Sep 26, 2023
Merged

Docs: adding veterans use case #1748

merged 10 commits into from Sep 26, 2023

Conversation

indexing
Copy link
Member

Closes #1587

@indexing indexing self-assigned this Sep 22, 2023
@indexing indexing requested a review from a team as a code owner September 22, 2023 22:40
@github-actions github-actions bot added the documentation [auto] Improvements or additions to documentation label Sep 22, 2023
docs/use-cases/Veterans.md Outdated Show resolved Hide resolved
This styling came with the .md file export from ClickUp. We definitely need to be aware that the Markdown in these exports isn't clean and should be reviewed carefully.
thekaveman
thekaveman previously approved these changes Sep 25, 2023
Copy link
Member

@thekaveman thekaveman left a comment

Choose a reason for hiding this comment

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

Nice work @indexing!

angela-tran
angela-tran previously approved these changes Sep 25, 2023
@thekaveman
Copy link
Member

thekaveman commented Sep 25, 2023

@machikoyasuda I think you can merge this once you've had a chance to review.

I'm guessing @indexing does not see the Merge pull request button because of permissions?

image

Copy link
Member

@machikoyasuda machikoyasuda left a comment

Choose a reason for hiding this comment

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

2 small suggestions @indexing (which you can just click Commit suggestion on) and then we're good!

@angela-tran
Copy link
Member

angela-tran commented Sep 26, 2023

From an earlier comment

@machikoyasuda I think you can merge this once you've had a chance to review.

I'm guessing @indexing does not see the Merge pull request button because of permissions?
image

@indexing Just curious, whenever the pull request is approved, do you see a "Merge pull request" button?

Co-authored-by: machiko <machiko@compiler.la>
Co-authored-by: machiko <machiko@compiler.la>
Copy link
Member Author

@indexing indexing left a comment

Choose a reason for hiding this comment

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

Accepted suggestions

docs/use-cases/Veterans.md Outdated Show resolved Hide resolved
Co-authored-by: Kegan Maher <kegan@compiler.la>
@thekaveman
Copy link
Member

@machikoyasuda go ahead and merge when you approve -- I confirmed with @indexing he does not currently have the permissions.

@machikoyasuda machikoyasuda merged commit a792131 into dev Sep 26, 2023
8 checks passed
@machikoyasuda machikoyasuda deleted the docs/veterans-use-case branch September 26, 2023 22:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation [auto] Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update docs site with Veteran use case
4 participants