Skip to content

WIP Android message table support #627

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.

Sign up for GitHub

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

Jump to bottom
Closed
dmose wants to merge 61 commits into from
Closed

WIP Android message table support #627

dmose wants to merge 61 commits into from

Conversation

@dmose
Copy link
Member

@dmose dmose commented Mar 18, 2025

No description provided.

@netlify
Copy link

netlify bot commented Mar 18, 2025 •
edited
Loading

✅ Deploy Preview for fxms-skylight ready!

Name Link
🔨 Latest commit 97c17ec
🔍 Latest deploy log https://app.netlify.com/sites/fxms-skylight/deploys/67eae058c54fd40008c1d961
😎 Deploy Preview https://deploy-preview-627--fxms-skylight.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse		 1 paths audited
Performance: 100 (🟢 up 3 from production)
Accessibility: 91 (🟢 up 2 from production)
Best Practices: 92 (🟢 up 9 from production)
SEO: 100 (🟢 up 10 from production)
PWA: -
View the detailed breakdown and full score reports

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

dmose added 29 commits March 26, 2025 10:05
@dmose
Initial TODO list
e4146bf
@dmose
Cleanup TODO list some
e42fc65
@dmose
Pull Dashboard into own component used by page.txt
9d2c9c5
@dmose
Remove unneeded import
0a48fa8
@dmose
Added launch config for jest tests
ee10c90
@dmose
Basic refactor without tests
6f871be
@dmose
Update TODO.md
73399b5
@dmose
Git rid of cruft and fix a test
05fa087
@dmose
Fix dashboard test
99feb17
@dmose
Wrap prodtable in a div in prep for hoisting
8d2e9b2
@dmose
Add android route
3f928d2
@dmose
Switch to JSX syntax
b7a16e2
@dmose
Factor out ReleaseTable into its own component
440be3e
@dmose
Appease prettier
1c981ee
@dmose
Appease prettier more
37b9767
@dmose
Comment out SurfaceTag use; will implement later
43a6449
@dmose
Make Dashboard platform prop required
aab841c
@dmose
Update + reformat TODO.md
76c9f1e
@dmose
Update TODO list
a94cc47
@dmose
Update roadmap
51e7302
@dmose
Cleanup more platform parameter lossage, partly
43fd27b
@dmose
Update TODO list
2d7265a
@dmose
Add basic instructions for copilot case sensitivity.
8e1cd01
@dmose
Refactor page.tsx to call fetchData and pass params
5766fa8
@dmose
Fixup Android calling of Dashboard component.
f7e8cf5
@dmose
Made localData optional to Dashboard component
e318946
@dmose
Make the file case instructions clearer
4613ad4
@dmose
Fixup dashboard test
ee64811
@dmose
Move platform to a union type
9d5d919
@sarahhjchung sarahhjchung mentioned this pull request Apr 1, 2025
Merged
@dmose
Copy link
Member Author

dmose commented Apr 8, 2025

Now that the first part (big page/dashboard/fetchData refactor) has landed (thanks @sarahhjchung!), closing this in favor of #653, which has the newer patches from here rebased on current main.

@dmose dmose closed this Apr 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@dmose