Skip to content

Conversation

@eshanrnh
Copy link
Contributor

@eshanrnh eshanrnh commented Apr 7, 2025

Description

... and replaced environment references

Type of suggestion

  • Typo/grammar fix
  • Updated outdated content
  • New content
  • Updates related to a new version
  • Other

Product & version (if relevant)

Cloud

Deadline (if relevant)

Anytime

Copy link
Contributor

@sofietoft sofietoft left a comment

Choose a reason for hiding this comment

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

I've added a couple of comments.

Instead of only having a heading for "Configuring an environment", wouldn't it make sense to have the following instead:

Adding an environment

Removing an environment

And these are added as subheadings to the ## Adding or Removing heading?

@eshanrnh
Copy link
Contributor Author

eshanrnh commented Apr 7, 2025

Thanks, @sofietoft for your review 🙌 I've incorporated your comments. Also, updated the Configuring Environments section to mention what actions are possible in that view.

Copy link
Contributor

@sofietoft sofietoft left a comment

Choose a reason for hiding this comment

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

Just one more thing, and this article is good to go 💪

@eshanrnh
Copy link
Contributor Author

eshanrnh commented Apr 8, 2025

Thanks for the review, @sofietoft 🙌 I've incorporated your comments and fixed a broken image filename in the Environments article.

@eshanrnh eshanrnh merged commit 891663d into cloud/flexibleEnvironment Apr 8, 2025
2 checks passed
@sofietoft sofietoft deleted the manage-environments branch April 28, 2025 07:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants