Skip to content

feat: web(for coding) #2088

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

Closed
wants to merge 0 commits into from
Closed

feat: web(for coding) #2088

wants to merge 0 commits into from

Conversation

Rutik7066
Copy link

@Rutik7066 Rutik7066 commented Mar 26, 2025

This PR aims to enable web support for the OMI flutter app.
fixes #2008
/claim #2008

Progress

@Rutik7066 Rutik7066 marked this pull request as ready for review March 26, 2025 19:04
@beastoin
Copy link
Collaborator

beastoin commented Mar 28, 2025

1/ reformat your code with char len 120 please
2/ #2008 (comment)

@Rutik7066

@Rutik7066
Copy link
Author

@beastoin For the correct direction. Do we want flutter app to work on web for editing/building as described in issue description or we want to add full support for web?

@Rutik7066
Copy link
Author

Rutik7066 commented Mar 29, 2025

Aah understood, First I will make ui responsive and adaptive. Then I will update to you about dependencies which doesn't support and what can be possible outcomes.

@Rutik7066
Copy link
Author

Here is the progress so far

output.mp4

@beastoin thanks! ffmpeg helped me 😊
Please let me know your thoughts

@beastoin
Copy link
Collaborator

man, good to see your progress btw, you should match(or do it better) the suggestion on the code first then update the UI/UX later if needed, or we just go with the dev tool - the enable mobile's size for example.

@Rutik7066

@Rutik7066
Copy link
Author

Rutik7066 commented Mar 31, 2025

Your right, as mentioned here
#2008 (comment) -2760090149 I started working on ui first and now few components remained. Once I complete Ui part probably by the end of the day. I will get to the platform specific code since I will need your help incase of dev kit related code

man, good to see your progress btw, you should match(or do it better) the suggestion on the code first then update the UI/UX later if needed, or we just go with the dev tool - the enable mobile's size for example.

@Rutik7066

@Rutik7066 Rutik7066 force-pushed the web branch 2 times, most recently from 5840bde to 2a0562f Compare April 1, 2025 14:18
@Rutik7066
Copy link
Author

Rutik7066 commented Apr 1, 2025

Hey, @beastoin Major packages which is not supported on web

Notifications

Currently Notification is not working fully on web. awesome_notifications are not supported on web. They are going to add support for web check https://pub.dev/packages/awesome_notifications#-next-steps for now we have added if check for web
flutter_foreground_task is also used for notification and not supported for web and not need as well so i added if checks for the code

Analytics and Support

intercom_flutter does not support the web check https://pub.dev/packages/intercom_flutter

Bluetooth

flutter_blue_plus Support some functionalty on web check https://pub.dev/packages/flutter_blue_plus#compatibility

We use

Method Description
setLogLevel Configure plugin log level
stopScan Stop an existing scan for BLE devices
mtuNow The current MTU value
readRssi Read RSSI from a connected device
requestMtu Request to change the MTU for the device

We can use js interop as alternative on web

File Operations

dart:io is not supported on web. We heavily use it for file handling.
consider alternative like in-memory database or data structures for web

@Rutik7066
Copy link
Author

@beastoin I wrapped all platform specific code and now I want your suggestion for bluetooth and file handling

@beastoin
Copy link
Collaborator

beastoin commented Apr 7, 2025

man, sry for the late response, will back to you next week.

btw, please make your PR super clean* and check all my suggestions (it will help your PR more valuable).

feel free to ask if you found any ambitious.

*: for example, i don't really understand why do you need these changes.

Screenshot 2025-04-07 at 14 59 15

@Rutik7066

@Rutik7066
Copy link
Author

Hey, @beastoin Major packages which is not supported on web

Notifications

Currently Notification is not working fully on web. awesome_notifications are not supported on web. They are going to add support for web check https://pub.dev/packages/awesome_notifications#-next-steps for now we have added if check for web flutter_foreground_task is also used for notification and not supported for web and not need as well so i added if checks for the code

Analytics and Support

intercom_flutter does not support the web check https://pub.dev/packages/intercom_flutter

Bluetooth

flutter_blue_plus Support some functionalty on web check https://pub.dev/packages/flutter_blue_plus#compatibility

We use

Method Description
setLogLevel Configure plugin log level
stopScan Stop an existing scan for BLE devices
mtuNow The current MTU value
readRssi Read RSSI from a connected device
requestMtu Request to change the MTU for the device
We can use js interop as alternative on web

File Operations

dart:io is not supported on web. We heavily use it for file handling. consider alternative like in-memory database or data structures for web

@beastoin No problem! I will clean up the PR. Please check this and let me know your thoughts. and I have sent you a message about split pay.

@Rutik7066
Copy link
Author

Rutik7066 commented Apr 7, 2025

I need to open a new PR since this PR is messed up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make omi work in browser (for coding)
2 participants