Skip to content

Minor changes in story_image #186

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.

Sign up for GitHub

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

Jump to bottom
Open
biswajit-synkrama wants to merge 61 commits into
base: master
Choose a base branch
from
Open

Minor changes in story_image #186

biswajit-synkrama wants to merge 61 commits into from

Conversation

@biswajit-synkrama
Copy link

@biswajit-synkrama biswajit-synkrama commented May 21, 2024

  • Change in story image common widget
  • Change the version of the story view package

Kashmira and others added 30 commits July 18, 2023 16:02
Support for HLS Video
a04743e
@cmile-kashmira-mayekar
Merge pull request #1 from KashmiraMayekar/kashmira/HLSVideo
237e392 
Support for HLS Video
Added Shimmer for Video and preloading story
d2c350d
@cmile-kashmira-mayekar
Merge pull request #2 from KashmiraMayekar/kashmira/HLSVideo
2cfeace 
Added Shimmer for Video and preloading story
Preloading issue
4265d62
@cmile-kashmira-mayekar
Merge pull request #3 from KashmiraMayekar/kashmira/HLSVideo
da4a0df 
Preloading issue
Remove Preloading
b62f027
@cmile-kashmira-mayekar
Merge pull request #4 from KashmiraMayekar/kashmira/HLSVideo
a8a7ccf 
Remove Preloading
Added Shimmer and Blur background
b3bdf14
@cmile-kashmira-mayekar
Merge pull request #5 from KashmiraMayekar/kashmira/HLSVideo
322ab2d 
Added Shimmer and Blur background
Remover blur backgroud
f1226b3
Merge remote-tracking branch 'origin/master'
12f4fa7 
# Conflicts:
#	lib/widgets/story_image.dart
Remover blur backgroud
a4380b0
@rutvikpatel30
Cached Player Implemented
a665c85
Basic Setup for Story viewing
af4edcc
Added storyId
8116a2c
Refactoring utils
51ef028
@synkrama-aditya-bhandare
Merge pull request #7 from cmile-kashmira-mayekar/aditya/StoryReply
9fa5eb9 
Aditya/story reply
Story index navigation
1839661
Merge remote-tracking branch 'origin/master'
483a0bd 
# Conflicts:
#	lib/controller/story_controller.dart
#	lib/widgets/story_view.dart
Minor Issue
cd5d181
Resolved issue
584a4a5
Story Index Navigation
3886c08
Story Index Navigation
05012d9
Story Index Navigation
f0e4952
Added repost story view
238be86
Minor changes
8a53d81
video story repost
a8bf238
Handled Null forvideo
614104b
Video issue resolved
b20f38f
Kashmira and others added 29 commits November 3, 2023 15:53
Minor changes
44f145e
Story view changes
a484325
minor changes
0732d60
minor changes
6f13e83
Added callback method
5ed19e6
View Post Method added
ba2cbe7
remove View Post
492b7ca
Addition of Repost flag onstory show
faab818
Is Repost Story flag added
95fe949
@cmile-kashmira-mayekar
Merge pull request #8 from cmile-kashmira-mayekar/aditya/StoryReply
4ac1304 
Aditya/story reply
Validate repost story
10472bd
Minor changes
1daa62d
Minor Changes
024ee9b
On click for view post
7a06b37
ISLike Functionality Added
89bb673
View post button added
aed1cc7
added padding
1c688eb
Added Margin
585e107
Remove video background
5ac832a
Remove Video Background
bcac7cf
Added random background too video
1c84636
Added boxfit to video
2f8f6c9
Minor Changes
4fe4257
minor changes
4d6a078
Remove LinearGradient
e30be17
LinearGradient
9f8319d
Added validation for view post button
2fd4bc4
Added loading condition for view post
0d18f5e
@biswajit-synkrama
minor changes done
981bbfc
geekpius
geekpius reviewed Jul 6, 2024
View reviewed changes
Copy link
Collaborator

@geekpius geekpius left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Please I can't merge this request. You have too many files touched. You can close this PR and break your PR into multiple PRs in order of significant

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

Reviewers

@geekpius geekpius geekpius left review comments

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@biswajit-synkrama @geekpius @cmile-kashmira-mayekar @rutvikpatel30 @synkrama-aditya-bhandare