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(inbound-email-connector): add inbound email connector documentation #4463

Merged
merged 9 commits into from
Nov 8, 2024

Conversation

mathias-vandaele
Copy link
Contributor

@mathias-vandaele mathias-vandaele commented Oct 15, 2024

Description

This feature will be available in the next connector patch 8.6.3

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/).

Copy link
Contributor

github-actions bot commented Oct 15, 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.

This inbound connector creates a new process each time a new email is received.

- `Folder`: (Optional) This parameter defines the folder that the inbound connector will monitor. If not specified, the default folder is set to INBOX.
- `Polling Wait Time`: This parameter sets the interval between each polling operation. Please [refer to the documentation](https://docs.camunda.io/docs/components/modeler/bpmn/timer-events/#time-duration) on time duration for the correct format.
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [vale] reported by reviewdog 🐶
[all.hrefProduction] Improper link format: 'refer to the documentation'. Please use relative URLs.

@christinaausley christinaausley added the component:connectors Issues related to the connectors project. label Oct 15, 2024
@christinaausley
Copy link
Contributor

default folder is set to INBOX. For subfolders, use `.` or `/` separated path (ex: `inside/folder` or
`inside.folder`)
- `Polling Wait Time`: This parameter sets the interval between each polling operation.
Please [refer to the documentation](https://docs.camunda.io/docs/components/modeler/bpmn/timer-events/#time-duration)
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [vale] reported by reviewdog 🐶
[all.hrefProduction] Improper link format: 'refer to the documentation'. Please use relative URLs.

@akeller
Copy link
Member

akeller commented Nov 4, 2024

What is the status of this PR?

sbuettner
sbuettner previously approved these changes Nov 5, 2024
@mathias-vandaele
Copy link
Contributor Author

Relates to camunda/product-hub#2430.

It will be released on 8.6.4 of connectors this week

@mesellings
Copy link
Contributor

@mathias-vandaele I assume this means this should be included in the alpha release next week then?

  • If so, please can you add the relevant label(s) if this is correct (for example, "8.7.0-alpha1") and complete the When should this change go live and PR checklist sections.
  • I can then plan and complete a TW review of this PR for you 👍

@mathias-vandaele mathias-vandaele added the 8.7.0-alpha1 November 2024 alpha release label Nov 7, 2024
@mathias-vandaele
Copy link
Contributor Author

@mathias-vandaele I assume this means this should be included in the alpha release next week then?

  • If so, please can you add the relevant label(s) if this is correct (for example, "8.7.0-alpha1") and complete the When should this change go live and PR checklist sections.
  • I can then plan and complete a TW review of this PR for you 👍

Hello @mesellings I added the correct label, I want to mention this will also be added in the next patch of 8.6.

This will be part of the next alpha but this feature will also be present on latest

@mesellings
Copy link
Contributor

Thanks @mathias-vandaele I'll review this for you now 👍

@mesellings mesellings self-requested a review November 7, 2024 14:13
@mesellings mesellings self-assigned this Nov 7, 2024
@mesellings
Copy link
Contributor

@mathias-vandaele I assume this means this should be included in the alpha release next week then?

  • If so, please can you add the relevant label(s) if this is correct (for example, "8.7.0-alpha1") and complete the When should this change go live and PR checklist sections.
  • I can then plan and complete a TW review of this PR for you 👍

Hello @mesellings I added the correct label, I want to mention this will also be added in the next patch of 8.6.

This will be part of the next alpha but this feature will also be present on latest

I assume this means we should also backport to the 8.6 docs? (just checking)

@akeller
Copy link
Member

akeller commented Nov 7, 2024

I've updated the PR description based on the following:

This will be part of the next alpha but this feature will also be present on latest

To me, this means it should be available for both the current docs (8.6) and future (/next/ or 8.7). @mathias-vandaele please use the template to help us understand how and when to review your changes.

mesellings
mesellings previously approved these changes Nov 8, 2024
Copy link
Contributor

@mesellings mesellings left a comment

Choose a reason for hiding this comment

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

@mathias-vandaele I've committed my edits to the branch, and backported to the 8.6 docs, happy to approve, good job! 👍 🚀

@mesellings
Copy link
Contributor

@mathias-vandaele Shall I merge and release this for you?

mesellings
mesellings previously approved these changes Nov 8, 2024
Copy link
Contributor

@mesellings mesellings left a comment

Choose a reason for hiding this comment

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

Fixed broken link and typo - approved 👍

Copy link
Contributor

@mesellings mesellings left a comment

Choose a reason for hiding this comment

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

Approved after link fix 👍

@mesellings mesellings merged commit 72a338f into main Nov 8, 2024
7 checks passed
@mesellings mesellings deleted the docs-add-inbound-email-connector-documentation branch November 8, 2024 10:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
8.7.0-alpha1 November 2024 alpha release component:connectors Issues related to the connectors project.
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

5 participants