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.

Sign up for GitHub

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

Jump to bottom

Allow scheduling messages #452

Closed
yesteph opened this issue Jun 27, 2024 · 1 comment
Closed

Allow scheduling messages #452

yesteph opened this issue Jun 27, 2024 · 1 comment
Assignees
@marboledacci

Comments

@yesteph
Copy link
Contributor

yesteph commented Jun 27, 2024

Describe Request:

Our CircleCI pipelines need to send Slack notifications after long period (about several days).
Using standard job with sleep X command for that is totally inefficient and will not work because of the different timeouts.

An alternative is the orb uses the scheduling message capability of Slack.

To implement that, the orb must support a new parameter post_at. If the this parameter is not null, the API must be chat.scheduleMessage instead of the current default one chat.postMessage.

Examples:

workflows:
  send-scheduled-notification:
    jobs:
      - notify:
          context: slack-secrets
          post_at: 1551891428

Supporting Documentation Links:

https://api.slack.com/messaging/sending#scheduling
yesteph pushed a commit to yesteph/slack-orb that referenced this issue Jun 27, 2024
@aviv-steyssier
feat: add scheduling messages (CircleCI-Public#452)
5e8abc9
@yesteph yesteph mentioned this issue Jun 27, 2024
Merged
yesteph pushed a commit to yesteph/slack-orb that referenced this issue Jul 19, 2024
@aviv-steyssier
fixup! feat: add scheduling messages (CircleCI-Public#452)
5395e3f
marboledacci pushed a commit that referenced this issue Sep 6, 2024
@yesteph @aviv-steyssier
feat: add scheduling messages (#452) (#453)
faa3a38 
* feat: add scheduling messages (#452)

* fixup! feat: add scheduling messages (#452)

---------

Co-authored-by: Stephane TEYSSIER <stephane.teyssier-ext@aviv-group.com>
@marboledacci marboledacci self-assigned this Sep 11, 2024
marboledacci added a commit that referenced this issue Sep 12, 2024
@marboledacci @yesteph @aviv-steyssier
feat: add scheduling messages (#453) (#457)
229bffc 
* feat: add scheduling messages (#452) (#453)

* feat: add scheduling messages (#452)

* fixup! feat: add scheduling messages (#452)

---------

Co-authored-by: Stephane TEYSSIER <stephane.teyssier-ext@aviv-group.com>

* Add test for the scheduled_offset_seconds option

* Use bash on scripts

* Fix command

* Update to shellcheck to bash

* ignore sed shellcheck

* Use sh on main

* Add two versions for date and debug to notify

* Add date for alpine

* remove debug

* Update test message for offset

* force usage of bash and update windows test to makes sense with thisg

* update auxiliar scripts to bash

* Restore bash for testing

* Add explicit shell again

* Set powerhsell

* Add exe

* set bash on job call

* Use bash on setting env

---------

Co-authored-by: stephane teyssier <stephane.teyssier@gmail.com>
Co-authored-by: Stephane TEYSSIER <stephane.teyssier-ext@aviv-group.com>
@marboledacci
Copy link
Contributor

This feature is added in version 4.14.0.
I will close the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@marboledacci marboledacci

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@yesteph @marboledacci