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

chore(ts): port sync_status.ts #1167

Merged
merged 1 commit into from
Feb 23, 2025
Merged

Conversation

pano9000
Copy link
Contributor

Hi,

this PR ports the sync_status.js file to TS.

I've added two Type Related TODO remarks as well for future consideration

@pano9000
Copy link
Contributor Author

need to fix the Bootstrap import issue, not sure why I didn't catch that build error it in my local environmemt

@pano9000 pano9000 force-pushed the chore_ts-port_sync_status branch from 89302fd to ec57810 Compare February 12, 2025 07:21
@pano9000 pano9000 closed this Feb 12, 2025
@pano9000 pano9000 reopened this Feb 12, 2025
@pano9000 pano9000 changed the title chore(ts): port sync_status.ts HOLD: chore(ts): port sync_status.ts Feb 12, 2025
@pano9000 pano9000 marked this pull request as draft February 13, 2025 09:27
@pano9000 pano9000 changed the title HOLD: chore(ts): port sync_status.ts chore(ts): port sync_status.ts Feb 13, 2025
@pano9000
Copy link
Contributor Author

once #1168 is merged, I will rebase this change on top of that :-)

@pano9000 pano9000 force-pushed the chore_ts-port_sync_status branch from ec57810 to 265786e Compare February 22, 2025 13:17
@pano9000 pano9000 marked this pull request as ready for review February 22, 2025 13:18
@pano9000 pano9000 marked this pull request as draft February 22, 2025 13:18
@pano9000 pano9000 force-pushed the chore_ts-port_sync_status branch from 265786e to 9fb9558 Compare February 22, 2025 13:22
@pano9000 pano9000 marked this pull request as ready for review February 22, 2025 13:22
@eliandoran eliandoran merged commit e5aed0a into develop Feb 23, 2025
5 checks passed
@eliandoran eliandoran deleted the chore_ts-port_sync_status branch February 23, 2025 16:26
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