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

chore: add mode and defaultStickiness to project docs #3487

Merged
merged 8 commits into from Apr 21, 2023

Conversation

andreas-unleash
Copy link
Contributor

Adds mode and defaultStickiness to the project form.
Adds a new img

About the changes

Closes #

Important files

Discussion points

Signed-off-by: andreas-unleash <andreas@getunleash.ai>
Signed-off-by: andreas-unleash <andreas@getunleash.ai>
@vercel
Copy link

vercel bot commented Apr 10, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
unleash-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 21, 2023 7:24am
1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
unleash-monorepo-frontend ⬜️ Ignored (Inspect) Visit Preview Apr 21, 2023 7:24am

@andreas-unleash andreas-unleash changed the title chore: fix docs chore: add mode and defaultStickiness to project docs Apr 10, 2023
Copy link
Contributor

@thomasheartman thomasheartman left a comment

Choose a reason for hiding this comment

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

So this looks pretty good to me! 😄 I've got two minor change suggestions that you can choose to accept or not.

There's only one thing that's keeping me from approving this:
It doesn't look like you've deleted the old image. Could you do that? I don't think it's being used anywhere else, is it? If it is, then ignore this 💁🏼

website/docs/reference/projects.md Outdated Show resolved Hide resolved
website/docs/reference/projects.md Outdated Show resolved Hide resolved
andreas-unleash and others added 2 commits April 20, 2023 22:57
Co-authored-by: Thomas Heartman <thomas@getunleash.ai>
Co-authored-by: Thomas Heartman <thomas@getunleash.ai>
Copy link
Contributor

@thomasheartman thomasheartman left a comment

Choose a reason for hiding this comment

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

Perfect! 🏆

@andreas-unleash andreas-unleash enabled auto-merge (squash) April 21, 2023 07:26
@andreas-unleash andreas-unleash merged commit b38a0c9 into main Apr 21, 2023
11 checks passed
@andreas-unleash andreas-unleash deleted the docs/update_project_with_mode_stickiness branch April 21, 2023 07:32
andreas-unleash added a commit that referenced this pull request Apr 24, 2023
<!-- Thanks for creating a PR! To make it easier for reviewers and
everyone else to understand what your changes relate to, please add some
relevant content to the headings below. Feel free to ignore or delete
sections that you don't think are relevant. Thank you! ❤️ -->
Adds `mode` and `defaultStickiness` to the project form.
Adds a new img

## About the changes
<!-- Describe the changes introduced. What are they and why are they
being introduced? Feel free to also add screenshots or steps to view the
changes if they're visual. -->

<!-- Does it close an issue? Multiple? -->
Closes #

<!-- (For internal contributors): Does it relate to an issue on public
roadmap? -->
<!--
Relates to [roadmap](https://github.com/orgs/Unleash/projects/10) item:
#
-->

### Important files
<!-- PRs can contain a lot of changes, but not all changes are equally
important. Where should a reviewer start looking to get an overview of
the changes? Are any files particularly important? -->

## Discussion points
<!-- Anything about the PR you'd like to discuss before it gets merged?
Got any questions or doubts? -->

---------

Signed-off-by: andreas-unleash <andreas@getunleash.ai>
Co-authored-by: Thomas Heartman <thomas@getunleash.ai>
(cherry picked from commit b38a0c9)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

3 participants