background_locator icon indicating copy to clipboard operation
background_locator copied to clipboard

Let's work all together on an other fork ?

Open Yukams opened this issue 2 years ago • 9 comments

Hi dear community,

I've been working with (or against) this repository for 2 months now. I worked on fixing it on my branch and succeeded to make it work in Android as expected (phone reboot, app killed, re-opening the app, support for Flutter 3.0, callback working properly) thanks to some of the pull requests open here.

Though, I still see that a lot of new people come across this repo and find themselves stuck with the same problems I've faced.

Let's be real, rekabhq, the owner of this library, will never update it again. We need to move onto someone else's fork. I personally can't figure out how to fix it for iOS, though I desperately need it.

As I'm a young developer, I'm pretty sure someone with open source experience would be better than me to maintain this project. If any of you feels like it, you can fork my fork of the library and we could try to all move to this new repo. Else we'll be stuck forever here and new people will keep losing their time on a broken lib.

Note : my fork is quite a mess when it comes to all the lasts commits, but hey, it works at least.

EDIT : We now have it on pub.dev ! https://pub.dev/packages/background_locator_2

Yukams avatar Jun 20 '22 08:06 Yukams

Hi @Yukams I think that is a good idea what you propose, after all thats the objetive of open source.

PD: I was looking at your package and I see that you solved the reboot issue, my only question is that I test trying to save the location updates in a database at the time of reboot and this is not doing it, would you know why? (That is, when you start the service and once you restart the cell phone, do you continue to receive updates?)

lasd14 avatar Jun 24 '22 18:06 lasd14

Hello @lasd14, I just tried it on my project and yes it's still working for me on Android after a phone reboot ! Just saying you have to restart the location simulation on your Android emulator after a phone reboot :)

Though, you can from now on publish issues and improvements directly on my fork :)

Yukams avatar Jun 27 '22 09:06 Yukams

Can you upload your git as a new project and plugin in pub dev? we all be thankful to you

daviddagan avatar Jul 07 '22 14:07 daviddagan

Hi @daviddagan, I'll see what I can do !

Yukams avatar Jul 08 '22 12:07 Yukams

hey I'm trying to use your fork and you forced the plugin to use min SDK version 21 it was necessary?.

daviddagan avatar Jul 10 '22 12:07 daviddagan

Hi @daviddagan, so I had a quick look and it seems like the lib would work with lower minSDK version (I tried with 19). I'm pretty sure I had a good reason to set it higher but couldn't find it today, maybe I don't have to anymore. I'll have a deeper look tomorrow, what minSDK do you need ? Also, feel free to open an issue directly on my fork's github page, as it may be better for tracking the evolution of said fork

Yukams avatar Jul 10 '22 12:07 Yukams

Hi @Yukams, the Min Sdk before has been 19. and sure, next time it will be on your forked git :)

daviddagan avatar Jul 10 '22 14:07 daviddagan

Hi @daviddagan, I've pushed a branch for a minsdk of 19, you can use it by putting this in your pubspec.yaml under dependancies

background_locator:
  git:
    url: https://github.com/Yukams/background_locator_fixed
    branch: minsdk_29

I'm looking right now to publish my fork on pub.dev too, I will update you on that

Yukams avatar Jul 11 '22 11:07 Yukams

EDIT : the fork is already up to date, ignore my previous message.

So I added the package to pub.dev https://pub.dev/packages/background_locator_2 Now you should be able to use it by adding this to your pubspec.yaml :

background_locator_2: ^2.0.3

Also you'll need to replace all your imports from background_locator to background_locator_2 All the rest should work the same

Yukams avatar Jul 11 '22 14:07 Yukams