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

Feature -> Develop | Added Platform Channel Service #44

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

we-mohd-i001
Copy link
Collaborator

Task #28 - Platform Specific Code > Method Channel & Event Channel

Time invested: hh:mm format 18:20 | Billable: 18:20 | Non-Billable: 00:00

Problem Statement

  1. The VaahFlutter doesn't have the option to use the Platform channels to establish communication between the native and Flutter side using Method and Event channels.

Describe The Fix/ Solution You Implemented
I have added a service that can used to invoke methods and get event streams from the native side.

Proof of your testing (Demo link or video links or image links)

Dependencies
Updated:
Added:

Merge Request Checklist

  • I have performed a self-review of my code
  • My code does follow industry standards
  • My code doesn't produce warnings/ errors for dart analyzer
  • New and existing tests pass locally with my changes (No tests are there as of now)
  • The code modified as part of this PR has been covered with tests
  • My code is properly formatted
  • I've followed the proper naming conventions
  • I have rebased the feature on the latest develop
  • I have run flutter run after the rebase
  • I have added new dependencies/ updated old dependencies
  • I have read all the comments & notes in wireframe and verified that I did take care of that
  • verify that the UI must match with design & wireframe if available
  • I have updated the version/ build (x.x.x+xxx)
  • I've verified that the latest commit of develop exists in your feature branch after rebase.
  • My fix achieves 100% of what is required
  • Can enhance the solution in the future because better implementation could be made

Added: platform base service

Added: implementation 

Added: singleton abstract class to use the service

Updated: return type to dynamic in invoke method

Added: generic type parameter
Added: methodChannelName parameter


Updated: method channel name from env config


Added: Exceptions 


Update: renamed MethodChannelService to PlatformChannelService


Added: MethodChannel parameter
Added: documentation comments for invoke method


Added: documentation comments for getEventStream


Updated: handling of other exceptions
@we-prajapati-c001
Copy link
Collaborator

Handing this over to @we-kislay-k001 from @we-mohd-i001

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.

3 participants