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

Sequence: send message to itself or polling #63

Open
lixueyi83 opened this issue Mar 29, 2023 · 4 comments
Open

Sequence: send message to itself or polling #63

lixueyi83 opened this issue Mar 29, 2023 · 4 comments
Assignees
Labels
enhancement New feature or request

Comments

@lixueyi83
Copy link

lixueyi83 commented Mar 29, 2023

Often times, I need to send a message to itself or polling and in which case I need to do A -> A: polling, something like this:

┌─┐
│A│
└┬┘
 │
 │Polling
 │-----+
 │     |
 │<----+
 │
┌┴┐
│A│
└─┘

However, it seems it does not work here:

$ diagon Sequence -- "A -> A: polling"
Something went wrong!
free(): invalid next size (normal)
Aborted
@ArthurSonzogni ArthurSonzogni self-assigned this Mar 29, 2023
@ArthurSonzogni ArthurSonzogni added the enhancement New feature or request label Mar 29, 2023
@ArthurSonzogni
Copy link
Owner

Yes. This is probably the top 1 feature I would personally need.

@ArthurSonzogni
Copy link
Owner

I wonder if we shouldn't also support ordering the start and the end.

Proposed syntax:

1,2) A -> A: Task 1
3,4) A -> A: Task 2

A: 1 < 3 < 2 < 4
┌─┐
│A│
└┬┘
 │
 │ Task 1
 │---------+
 │         |
 │ Task 2  |
 │------------+
 │         |  |
 │         |  |
 │<--------+  |
 │            |
 │<-----------+
 │
┌┴┐
│A│
└─┘

@lixueyi83
Copy link
Author

That would be highly appreciated, and by the way, I just started using this fancy tool and it's really so fancy! I like everything to be simple, easy and does the job perfectly! Thank you very much!

@bjornstromberg
Copy link

just found this tool, it sure does seem to do most of the things i would need of a Sequence Diagram tool,
from a quick look this feature and a way to make a visual difference between async and sync calls is whats missing,
but i'll create another issue for that one.

keep up the great work @ArthurSonzogni

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants