Skip to content

Amalesh - Create Refresh Button for Disconnected Timer#4459

Closed
Ramaruva wants to merge 4 commits intodevelopmentfrom
Ramakrishna_feat_timer_refresh
Closed

Amalesh - Create Refresh Button for Disconnected Timer#4459
Ramaruva wants to merge 4 commits intodevelopmentfrom
Ramakrishna_feat_timer_refresh

Conversation

@Ramaruva
Copy link
Copy Markdown
Contributor

@Ramaruva Ramaruva commented Nov 26, 2025

Description

image

Related PRS (if any):
This frontend PR is related to the development branch of the backend.

Main changes explained:

  • src/components/Timer/Timer.jsx
  • src/components/Timer/Timer.module.css
  • src/components/Timer/TimerStatus.jsx

How to test:

Method 1:

  1. check into current branch
  2. do npm install and ... to run this PR locally
  3. Clear site data/cache
  4. log in as a user
  5. Go to Network tab in the Browser DevTools
  6. Enable Request Blocking:
    • Click the (three dots) menu in Network tab
    • Select "More tools""Request blocking"
    • Click "+ Add pattern"
    • Enter pattern: *timer-service*
    • Enable the checkbox next to the pattern
  7. Refresh the page
  8. Verify Disconnected State:
    • The timer preview area should show "Disconnected" in red text
    • A refresh icon (circular arrows) should appear next to "Disconnected"
  9. Test Reconnection:
    • Unblock the timer-service pattern (uncheck the checkbox in Request Blocking)
    • Click the refresh icon
    • Wait a few seconds
    • Timer should reconnect and show the time display (e.g., "00:15:00")
    • The "Disconnected" message should disappear

Method 2:

  1. If the above method doesn't work:
    • Keep the app loaded on the page with the timer visible
    • Turn off Wi-Fi without refreshing the page
    • Wait a few seconds
  2. Verify Disconnected State:
    • The timer preview area should show "Disconnected" in red text
    • A refresh icon (circular arrows) should appear next to "Disconnected"
    • A side panel message should appear saying the connection with the timer service is closed
  3. Test Reconnection:
    • Turn Wi-Fi back on
    • Wait a few seconds for the connection to return
    • Click the refresh icon
    • Wait a few seconds
    • Timer should reconnect and show the time display again (for example, "00:15:00")
    • The "Disconnected" message should disappear

Screenshots or videos of changes:

PR.4459.mp4

Note:

Include the information the reviewers need to know.

@netlify
Copy link
Copy Markdown

netlify Bot commented Nov 26, 2025

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 43c185f
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/69bd4bcfe3f4650008e9cc4a
😎 Deploy Preview https://deploy-preview-4459--highestgoodnetwork-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@Ramaruva Ramaruva added the Needs New Developer This is a PR that is partially developed but needs someone new to take it over and finish it. label Nov 26, 2025
@one-community one-community changed the title disconnect refresher inital work Ramakrishna - disconnect refresher inital work Nov 27, 2025
@Amalesh-A Amalesh-A added do not review Do not review or look at code without full context and removed Needs New Developer This is a PR that is partially developed but needs someone new to take it over and finish it. labels Dec 13, 2025
@Amalesh-A Amalesh-A changed the title Ramakrishna - disconnect refresher inital work Amalesh - disconnect refresher inital work Dec 13, 2025
@Amalesh-A Amalesh-A changed the title Amalesh - disconnect refresher inital work Amalesh - Create Refresh Button for Disconnected Timer Dec 13, 2025
@Amalesh-A Amalesh-A force-pushed the Ramakrishna_feat_timer_refresh branch from 2bafe07 to 54640dc Compare December 14, 2025 02:59
@sonarqubecloud
Copy link
Copy Markdown

@Amalesh-A Amalesh-A added High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible and removed do not review Do not review or look at code without full context labels Jan 8, 2026
Copy link
Copy Markdown

@Anusha-Gali Anusha-Gali left a comment

Choose a reason for hiding this comment

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

Hi Amalesh,

I have reviewed your PR locally and unfortunately the timer did not stop even though i followed all the instructions mentioned.

4459.mov

Copy link
Copy Markdown

@HemanthNidamanuru HemanthNidamanuru left a comment

Choose a reason for hiding this comment

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

Hi Amalesh,

I reviewed your PR locally and followed the steps outlined in the description. However, the timer still continues to run and does not stop as expected.

Image Image

@Amalesh-A
Copy link
Copy Markdown
Contributor

Hello @Anusha-Gali and @HemanthNidamanuru! I’ve updated the instructions. Could you please review them and let me know if any further changes are needed? Thanks!

@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown

@HemanthNidamanuru HemanthNidamanuru left a comment

Choose a reason for hiding this comment

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

Hi Amalesh,

I reviewed your PR locally and followed the second method outlined in the description. However, when Wi-Fi is turned off, the timer does not show the "Disconnected" state and continues to run. The red "Disconnected" text, refresh icon, and side panel message do not appear as expected.

Image Image

Copy link
Copy Markdown

@Anusha-Gali Anusha-Gali left a comment

Choose a reason for hiding this comment

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

Hi Amalesh,

I tried your method 2 but it does not seem to work when network is offline as well as when wifi is disconnected. You can also observe the time difference between screenshot taken and the the time in video to see that no changes are observed.

Image Image
4459.mp4

@Amalesh-A Amalesh-A closed this Apr 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants