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

Update the Spring developer guide #4301

Conversation

camundom
Copy link
Contributor

@camundom camundom commented Sep 13, 2024

Description

Improved the guide to align with the upcoming release of Camunda 8 Desktop Modeler and C8 Run. This includes revising the Prerequisites section, removing redundant steps now covered by the new Self-managed setup, and improving the clarity and structure of the steps for getting started.

What's changed?

  • Prerequisites

    • Includes links to the OpenJDK distribution for easy access.
    • Includes links to download the Desktop Modeler & C8 Server
    • Helpful tip that informs the user if they have already taken the survey to look for the Download links in their email.
  • Discarded Step 1: Install Camunda 8 Self-managed

    • Removed this step entirely as it’s now redundant with the introduction of C8 Run
  • Steps 2 & 3

    • Updated to mention the latest Maven version - 3.3.3
    • Improved discoverability of the Properties panel and readability
    • Added link to ‘What is FEEL?’

When should this change go live?

  • This is a bug fix, security concern, or something that needs urgent release support.
  • This is already available but undocumented and should be released within a week.
  • This on a specific schedule and the assignee will coordinate a release with the DevEx team. (apply hold label or convert to draft PR)
  • This is part of a scheduled alpha or minor. (apply alpha or minor label)
  • There is no urgency with this change and can be released at any time.

PR Checklist

  • My changes are for an already released minor and are in /versioned_docs directory.
  • My changes are for the next minor and are in /docs directory (aka /next/).

Improved the guide to align with the upcoming release of Camunda 8 Desktop Modeler and C8 Run. Includes revising the Prerequisites section, removing redundant steps now covered by the new Self-managed setup, and improving the clarity and structure of the getting started steps.
@camundom camundom added hold This issue is parked, do not merge. target:8.6 Issues included in the 8.6 release labels Sep 13, 2024
Copy link
Contributor

github-actions bot commented Sep 13, 2024

👋 🤖 ✅ Looks like the changes were ported across versions, nice job! 🎉

You can read more about the versioning within our docs in our documentation guidelines.

Copy link
Collaborator

@pepopowitz pepopowitz left a comment

Choose a reason for hiding this comment

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

Yay, C8RUN!!!!! Thanks for updating this!!!

I think my one comment will result in changes one way or another.

Comment on lines 39 to 47
[comment]: SmPrereqs/

## Step 1: Install Camunda 8 Self-Managed
## Prerequisites

<Install/>
Before getting started, ensure you have the following in your local environment:

## Step 2: Create a new Spring Boot project
- Access to your preferred code editor or IDE
- [OpenJDK 21+](https://jdk.java.net/)
- [Camunda 8 Desktop Modeler](https://camunda.com/download/modeler/) and [Camunda 8 Server<br/>](https://camunda.com/download/modeler/)_If you've already signed up for a Self-managed account, you can find the download links in your welcome email._
Copy link
Collaborator

@pepopowitz pepopowitz Sep 16, 2024

Choose a reason for hiding this comment

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

blocking, in one way or another:

Previous to this, we've been creating a separate prerequisites sub-document for each different form of installation. I can get on-board with eliminating the sub-documents, especially if this is the final form of this guide. But if this isn't the final expected form of this guide, we should stick with that approach.

If we do stick with this approach of pulling this section inline instead of importing from a sub-document, then there are a couple things we should clean up:

  1. The [comment]: SmPrereqs/ line -- it's unclear to me why we would preserve that path as a "comment".
  2. The now-unused import lines at the top of this file:
    import SmPrereqs from './react-components/sm-prerequisites.md'
    import Install from './react-components/install-plain-java.md'
    
  3. (maybe) The unused sub-documents in the react-components folder.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for taking a look and sharing your feedback @pepopowitz !

This would be the final form of this guide. Would it be good to add these steps to the separate prerequisites sub-document instead, replacing the older format of installing SM? AFAIK, there are two other guides that use prerequisites and the sub-document would help the other guides as well.

The [comment]: SmPrereqs/ line -- it's unclear to me why we would preserve that path as a "comment".

This is temporary. If we go ahead with using the sub-document, then this line would get uncommented.

The now-unused import lines at the top of this file:

These would be removed as well.

PS: This is my first contribution towards docs where I've edited/written using markdown and understanding the structure was new to me :)

Copy link
Collaborator

Choose a reason for hiding this comment

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

Ah, I didn't realize there was reuse across guides -- we should definitely update in the sub-documents instead.

Thank you!!!

Copy link
Contributor

Choose a reason for hiding this comment

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

There is a component for reusing the C8Run installation content created already, but I haven't yet dropped it into existing pages like this. I would recommend using a separate component than the ones that already exist for the Java install, though, as I am not sure each page is ready to use with C8Run.

@akeller
Copy link
Member

akeller commented Sep 23, 2024

FYI @conceptualshark


## Step 2: Create a new Spring Boot project
- Access to your preferred code editor or IDE
- [OpenJDK 21+](https://jdk.java.net/)
Copy link
Contributor

@conceptualshark conceptualshark Sep 24, 2024

Choose a reason for hiding this comment

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

Do we want to be opinionated about this link? I opted against including a specific link/recommendation in the C8Run documentation, as each user may have a different preference for install (and using the java-provided install is More Annoying in some cases).

Copy link
Member

Choose a reason for hiding this comment

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

I want to be more opinionated, but my first concern is this is the wrong link to OpenJDK. We want something like https://openjdk.org/install/

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The idea in providing a direct link was to make it easier for users to quickly access the right resource—the same link OpenJDK provides on their installation page. We also received user feedback requesting direct links for easier access. The link to https://openjdk.org/install/ works fine as well.

Copy link
Contributor

Choose a reason for hiding this comment

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

I don't really love the OpenJDK install page - I think it's a little misleading/confusing, and OpenJDK offers no follow-up on setting JAVA_HOME when installed in this way.

Using homebrew or apt-get for mac and linux installs is going to be a lot more straightforward, but this opens us up to needing to document multiple paths of entry because OpenJDK's documentation isn't terribly robust. I just want to make sure that being opinionated doesn't actually get us fielding more questions than it answers.

Copy link
Contributor

Choose a reason for hiding this comment

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

Either way, a note to make sure JAVA_HOME is set correctly would probably help; that way, no matter the installation, that check happens before they continue.

## Step 2: Create a new Spring Boot project
- Access to your preferred code editor or IDE
- [OpenJDK 21+](https://jdk.java.net/)
- [Camunda 8 Desktop Modeler](https://camunda.com/download/modeler/) and [Camunda 8 Server<br/>](https://camunda.com/download/modeler/)_If you've already signed up for a Self-Managed account, you can find the download links in your welcome email._
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm unsure of the timeline for this PR, so some of these questions depend on if it's meant to go live before or after the upcoming marketing website changes to support SM/C8Run:

  1. We're using using the name Camunda 8 Run currently; should we use that name here for now, and update it if it's changed to "Camunda 8 Server"?

  2. While the marketing pages are still in development, we can link to the docs at https://docs.camunda.io/docs/next/self-managed/setup/deploy/local/c8run/

  3. Should the email download line be removed until that new flow is enabled?

Copy link
Member

Choose a reason for hiding this comment

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

The name will be Camunda 8 Run. After our meeting today, @melanie-butcher will communicate this to the respective teams.

@camundom, is your timeline for this PR aligned with the 8.6? Or just the marketing pages/assets being ready too?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@akeller, thanks for the update! Yes, the timeline for this PR is 8.6. The marketing pages are ready for preview and undergoing testing, while the download email is WIP.

@akeller akeller added the deploy Stand up a temporary docs site with this PR label Sep 24, 2024
@akeller
Copy link
Member

akeller commented Sep 24, 2024

After the prj-dev-gettingstarted meeting today, I made the following changes:

  • Removed the note referring to the tutorial only being for non-prod
  • Adjusted the 2nd note about greenfield - greenfield is not a term that resonates well with our users based on user testing, so we removed it

@github-actions github-actions bot temporarily deployed to camunda-docs September 24, 2024 22:19 Destroyed
@github-actions github-actions bot temporarily deployed to camunda-docs September 25, 2024 12:45 Destroyed
@conceptualshark
Copy link
Contributor

@akeller What led to removing the non-production notice? Is that something we'd need to repeat in other guides?

@akeller
Copy link
Member

akeller commented Sep 25, 2024

@akeller What led to removing the non-production notice? Is that something we'd need to repeat in other guides?

User feedback/testing - @camundom, this would be outside the scope of this PR, but do you have a recommendation for our other guides?

- Removed links to unused components
- Added note to ensure JAVA_Home is set properly
- Changed OpenJDK install url
@github-actions github-actions bot temporarily deployed to camunda-docs September 27, 2024 15:49 Destroyed
@conceptualshark
Copy link
Contributor

@camundom Is this still waiting for marketing/email journey updates? Is there a version of this update we can deliver for 8.6?

<Install/>
- Access to your preferred code editor or IDE
- [OpenJDK 21+](https://openjdk.org/install/)
- [Camunda 8 Desktop Modeler](https://camunda.com/download/modeler/) and [Camunda 8 Run<br/>](https://camunda.com/download/modeler/)_If you've already signed up for a Self-Managed account, you can find the download links in your welcome email._
Copy link
Contributor

Choose a reason for hiding this comment

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

I noticed that C8Run is linked to the modeller link.
Could we add a link to C8Run installation page?
I would not expect users to wait for email or look for email when they are already committed to trying out this guide.
Plus, we do not get the download link in our email. It is not as easy as downloading an app. it needs a few more steps.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It looks like the download emailer hasn't gone up yet. I'll add the link to the C8 Page.

@akeller akeller removed the deploy Stand up a temporary docs site with this PR label Oct 7, 2024
@akeller
Copy link
Member

akeller commented Oct 7, 2024

Removing deploy since it's currently broken. 😢

@akeller akeller added the deploy Stand up a temporary docs site with this PR label Oct 7, 2024
@akeller
Copy link
Member

akeller commented Oct 7, 2024

I resolved the merge conflicts that may not be in the spirit of this PR, but I wanted to get the deploy tag working again, which required merging in changes from main. @camundom please have a look.

@github-actions github-actions bot temporarily deployed to camunda-docs October 7, 2024 16:03 Destroyed
@akeller
Copy link
Member

akeller commented Oct 8, 2024

@camundom is this ready to merge? It will need to be backported to appear in the 8.6 docs, but it's something DevEx can help with. Just let us know.

@camundom
Copy link
Contributor Author

camundom commented Oct 8, 2024

@camundom is this ready to merge? It will need to be backported to appear in the 8.6 docs, but it's something DevEx can help with. Just let us know.

@akeller this is ready to merge

@camundom camundom removed the hold This issue is parked, do not merge. label Oct 8, 2024
@christinaausley
Copy link
Contributor

I will backport and merge this now.

@christinaausley christinaausley enabled auto-merge (squash) October 8, 2024 13:48
<Install/>
- Access to your preferred code editor or IDE
- [OpenJDK 21+](https://openjdk.org/install/)
- Camunda 8 Desktop Modeler and Camunda 8 Run
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we link to the docs for both here?

Copy link
Contributor

Choose a reason for hiding this comment

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

I will incorporate this in a separate PR @conceptualshark 👍

@christinaausley christinaausley merged commit 8840c28 into main Oct 8, 2024
7 checks passed
@christinaausley christinaausley deleted the 4300-update-the-spring-developer-guide-to-include-the-new-installation-of-self-managed branch October 8, 2024 14:10
@github-actions github-actions bot temporarily deployed to camunda-docs October 8, 2024 14:10 Destroyed
Copy link
Contributor

github-actions bot commented Oct 8, 2024

🧹 Preview environment for this PR has been torn down.

Copy link
Contributor

github-actions bot commented Oct 8, 2024

The preview environment relating to the commit f95a9cb has successfully been deployed. You can access it at https://preview.docs.camunda.cloud/pr-4301/index.html

@christinaausley christinaausley mentioned this pull request Oct 8, 2024
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deploy Stand up a temporary docs site with this PR target:8.6 Issues included in the 8.6 release
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

Update the Spring developer guide to include the new installation of Self-Managed
6 participants