Fixed issue #5553 : Warning message before unsubscribing from a stream

[arijitghosal03]

No description provided.

[chrisbobbe]

Hi! Could you please say more about the reason for this proposed change? 🙂

[arijitghosal03]

Hi @chrisbobbe , I was working on the issue #5553 and found that (StreamSettingsScreen.js) no alert message before unsubscribing from a Stream. My idea was to add an alert component where is the user chooses the ("OK") button, the execution gets directed to Unsubscribe function where his subscription gets cancelled. The above mentioned changes has been done in the JavaScript file

[gnprice]

Hi @chrisbobbe , I was working on the issue #5553 and found that (StreamSettingsScreen.js) no alert message before unsubscribing from a Stream. My idea was to add an alert component where is the user chooses the ("OK") button, the execution gets directed to Unsubscribe function where his subscription gets cancelled. The above mentioned changes has been done in the JavaScript file

It sounds like you're perhaps describing your other PR #5578.

This PR has just one commit, aec646d. It makes one change:

--- .github/workflows/ci.yml
+++ .github/workflows/ci.yml
@@ -6,7 +6,7 @@ jobs:
   test:
     runs-on: ubuntu-latest
     steps:
-    - uses: actions/checkout@v2
+    - uses: actions/checkout@v3
 
     - name: Set up JDK
       uses: actions/setup-java@v1

Can you say more about what led you to that change? Perhaps it'd be good to update that dependency, but I haven't noticed anything about it to draw my attention.