-
Notifications
You must be signed in to change notification settings - Fork 25
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
Load feed #54
Load feed #54
Conversation
Kindly add a video description to showcase the interaction of the UI concerning this feature. |
Okay
…On Thu, Aug 8, 2024, 9:53 PM Miller Adulu ***@***.***> wrote:
@DevshiDeno <https://github.com/DevshiDeno>
Kindly add a video description to showcase the interaction of the UI
concerning this feature.
—
Reply to this email directly, view it on GitHub
<#54 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AYQBRLZQXSMTN6GJKI764PDZQO5CNAVCNFSM6AAAAABMG2XQE2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENZWGQ2TSMBWGA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please delete this entire file. It's not needed.
Future<void> fetchFeeds() async { | ||
emit(const FetchFeedState.loading()); | ||
try { | ||
final feeds = await _apiRepository.fetchFeeds(event: ''); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The slug for the feed of this event as per the documentation at this URL should be droidconke-2022-281
https://documenter.getpostman.com/view/3385291/SzS4TTXb#a443ec01-1cf8-41d9-bfd5-56fb6c228de4
Please make this slight addition.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Delete this file.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Delete this file.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Delete this file.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Delete this file.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Delete this file.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Delete this file.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Delete this file.
Once you are done with the changes, check the failing pipeline here: https://github.com/droidconKE/flutterconKEApp/actions/runs/10305939605/job/28533957201?pr=54 Fix the formatting issue by running the |
The code generally looks OK. Looking forward to the corrections and this can be merged in quickly. |
Trusting that you are still on this. |
Yes, I am working on it.
…On Sun, Aug 11, 2024, 2:05 PM Miller Adulu ***@***.***> wrote:
@DevshiDeno <https://github.com/DevshiDeno>,
Trusting that you are still on this.
—
Reply to this email directly, view it on GitHub
<#54 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AYQBRL6JVOCLE37VNPXRVDTZQ5AO5AVCNFSM6AAAAABMG2XQE2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOBSG4YTSOBSGI>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Made changes on #59 that's now merged. Closing |
Description
load feed issue
I created the necessary models required to handle event feeds gotten from the API and used Cubit to track state on the feed UI screen.
Closes #48
Type of Change
enhancement
Test Changes on Device
You can download your APK from Firebase App Distribution once this PR builds successfully via the following link: https://appdistribution.firebase.dev/i/c796669942f8a811