Added ability to define transition type when defining route

[jonsamwell]

Also updated the test dependency in pubspec.yaml.

This is just a first pass to get your thoughts...

It should cover the enhancements needed for #27

[codecov-io]

Codecov Report

Merging #63 into master will decrease coverage by <.01%.
The diff coverage is 45.45%.

@@            Coverage Diff             @@
##           master      #63      +/-   ##
==========================================
- Coverage   50.28%   50.28%   -0.01%     
==========================================
  Files           3        3              
  Lines         173      175       +2     
==========================================
+ Hits           87       88       +1     
- Misses         86       87       +1

Impacted Files	Coverage Δ
lib/src/common.dart	44.44% <100%> (+6.94%)	⬆️
lib/src/router.dart	7.35% <33.33%> (-0.11%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 79e35c5...45b607f. Read the comment docs.

[jonsamwell]

Any update on this?

[lukef]

There are a couple of things:

1. It looks to be broken. I just ran the example app and the transitions are all executing the "native" transition.
2. Any transition passed via navigateTo should always take priority over anything defined at the route level (doesn't seem the case now)

I think there are a few other changes needed until this can be merged. For example: You will probably want to eliminate the default on the optional transition for navigateTo and check for null later otherwise you wont be able to test to see if it has been set to override.

You probably also want to make transition optional on the route with null as the default value.

[lukef]

I've done all the fixes here: https://github.com/lukef/fluro/tree/lf-route-trans. Want to take a look? It needs testing before I can merge.

[jonsamwell]

@lukef You branch looks good many thanks. I'll clone it and test.

[lukef]

Let me know when you think it's ready and I'll smoke test and we can be good to go.

[ghost]

Thanks all !

[jonsamwell]

@lukef I'm really sorry for the insane delay getting back to you!

I've just had a play with your changes for this and it is working really well. Thanks again for the changes.

I realize this is a couple of releases behind the main repo so let me know if you want me to do a PR to update your fork of this so we can merge.

Thanks again!

[Benoss]

This look awesome, can we get a release with that?

[jonsamwell]

@Benoss it is awesome! I'm just waiting on @lukef to see if he wants me to update his fork of this PR so it can be merged into the main branch without any conflicts.

[lukef]

If you can resolve the conflicts I'll do a release

[jonsamwell]

@lukef this PR is now updated from your working fork and the changes from master have also been merged in. I also added a line to the readme file to indicate how to specify a route transition when defining a route. Hopefully this is good to go now 👍

[jonsamwell]

@lukef any chance we can get this merged in and released?

[lukef]

1.4.0 is published