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

Send plugin notification when user selects new spool from UI #15

Merged

Conversation

gdombiak
Copy link

OctoPod app has support for this plugin. Noticed that plugin was not sending a websocket notification when selecting a spool. This PR adds this notification

@stale
Copy link

stale bot commented Nov 24, 2020

This issue has been automatically marked for closing, because it has not had activity in 30 days. It will be closed if no further activity occurs in 10 days.

@stale stale bot added the status: markedForAutoClose Issue will be closed automatically label Nov 24, 2020
@gdombiak
Copy link
Author

Hi @OllisGit, pinging you before the bot decides to close this PR! :)

@stale stale bot removed the status: markedForAutoClose Issue will be closed automatically label Nov 24, 2020
@stale
Copy link

stale bot commented Dec 24, 2020

This issue has been automatically marked for closing, because it has not had activity in 30 days. It will be closed if no further activity occurs in 10 days.

@stale stale bot added the status: markedForAutoClose Issue will be closed automatically label Dec 24, 2020
@gdombiak
Copy link
Author

gdombiak commented Dec 24, 2020

Hi @OllisGit, last try to keep this PR alive before the "evil" bot :) closes it for me! :)

@stale stale bot removed the status: markedForAutoClose Issue will be closed automatically label Dec 24, 2020
@OllisGit OllisGit changed the base branch from master to feature/15-spool-selection-notification January 12, 2021 16:41
@OllisGit OllisGit changed the base branch from feature/15-spool-selection-notification to master January 12, 2021 16:42
@OllisGit
Copy link
Owner

Sorry, for being late to the party, but I made a "Octoprint sabatical" (no coding, no printing) the last months.

MR is merged and will be in the next release!

@OllisGit OllisGit added status: inNextRelease Will be implemented/fixed in next release and removed status: analysing labels Jan 28, 2021
@OllisGit OllisGit merged commit cc108c3 into OllisGit:master Jan 28, 2021
OllisGit added a commit that referenced this pull request Jan 28, 2021
OllisGit added a commit that referenced this pull request Jan 28, 2021
- MR #15 inform external ui (like OctoDash) about filament change. Credit goes to @gdombiak
- MR #28 creating and selecting a spool via API. Credit goes to @oschwartz10612
OllisGit added a commit that referenced this pull request Jan 28, 2021
- MR #15 inform external ui (like OctoDash) about filament change. Credit goes to @gdombiak
- MR #28 creating and selecting a spool via API. Credit goes to @oschwartz10612
@OllisGit OllisGit removed the status: inNextRelease Will be implemented/fixed in next release label Jun 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants