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

Added interceptor's #674

Merged
merged 14 commits into from
Oct 16, 2024
Merged

Added interceptor's #674

merged 14 commits into from
Oct 16, 2024

Conversation

Sten435
Copy link
Contributor

@Sten435 Sten435 commented Aug 4, 2024

Added BeamInterceptor and fixed a BeamPage.noTransition issue and general improvements

stan-at-work and others added 9 commits July 23, 2024 14:00
…lways return.

If there are 4 path pattern's and all of them are regex, then there was a bug that only the first one was checked. if that one failed then all the other ones where skipped. this fixes that.
…l-checked

Fix regexp path patern not all checked
…ionDuration where not set to Duration.zero. And thus the default dustion of 300 milliseconds was used

Add the option to pass transitionDuration, ans reverseTransitionDuration to beamPage. Fixed an issue where the duration of BeamPageType.noTransition was not zero.
…-general-improvements

Added interceptor feature and general improvements
…-general-improvements

Fixed linter issue
Copy link
Owner

@slovnicki slovnicki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see a lot of diffs which are just formatted differently.
Can you run dart format package/lib so we get rid of those?

@stan-at-work
Copy link
Contributor

Done ✅

Copy link

codecov bot commented Aug 6, 2024

Codecov Report

Attention: Patch coverage is 74.46809% with 36 lines in your changes missing coverage. Please review.

Project coverage is 93.45%. Comparing base (1649a65) to head (8c7168b).

Files Patch % Lines
package/lib/src/beam_interceptor_pop_scope.dart 0.00% 15 Missing ⚠️
package/lib/src/beamer_delegate.dart 45.83% 13 Missing ⚠️
package/lib/src/beam_interceptor.dart 0.00% 6 Missing ⚠️
package/lib/src/beamer.dart 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #674      +/-   ##
==========================================
- Coverage   97.08%   93.45%   -3.63%     
==========================================
  Files          14       16       +2     
  Lines         893      948      +55     
==========================================
+ Hits          867      886      +19     
- Misses         26       62      +36     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sten435 pushed a commit to stan-at-work/beamer that referenced this pull request Oct 8, 2024
@Sten435 Sten435 changed the title Added interceptor feature and general improvements Added interceptor's Oct 8, 2024
@slovnicki slovnicki merged commit df074ef into slovnicki:master Oct 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants