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(web-modeler): remove unnecessary links from publish process options #4998

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

huygur
Copy link
Contributor

@huygur huygur commented Feb 13, 2025

Description

I came across this page and noticed the url list looks a bit weird, and I thought this was not intentional. If I am mistaken, just ignore and close my PR 😄

When should this change go live?

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

PR Checklist

  • My changes are for an upcoming minor release and:
    • are in the /docs directory (version 8.8).
    • are in the /versioned_docs/version-8.7/ directory (version 8.7).
  • My changes are for an already released minor and are in a /versioned_docs directory.

@huygur huygur marked this pull request as ready for review February 13, 2025 13:21
@huygur huygur requested a review from wollefitz February 13, 2025 13:21
@akeller akeller added component:web-modeler-SaaS Continuous delivery - Issues related with Web Modeler SaaS component:web-modeler-sm Issues related with Web Modeler Self-Managed low prio There is no urgency with this change. labels Feb 13, 2025
@akeller akeller requested a review from christinaausley March 3, 2025 15:55
@akeller
Copy link
Member

akeller commented Mar 3, 2025

@christinaausley is this maybe from your VSCode plugin? Can you review this on behalf of the Modeler team (or pull them in if it's something you need help with)?

@christinaausley
Copy link
Contributor

@akeller Could've been, but I don't think I have edited this page in awhile. I have since disabled the plugin, so hopefully we don't see many of these moving forward.

Thank you @huygur for flagging this and opening a PR -- looks fine to me, but will leave the final say up to @wollefitz 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:web-modeler-SaaS Continuous delivery - Issues related with Web Modeler SaaS component:web-modeler-sm Issues related with Web Modeler Self-Managed low prio There is no urgency with this change.
Projects
Status: 👀 In Review
Development

Successfully merging this pull request may close these issues.

3 participants