-
Couldn't load subscription status.
- Fork 0
Build/update dependencies #107
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
Conversation
- Update auth-related packages from v1.0.0 to v1.0.1 - Update core package from v1.2.0 to v1.3.1 - Update data-related packages from v1.0.0 to v1.0.1 - Update bloc from v9.0.0 to v9.1.0 - Update go_router from v16.2.2 to v16.3.0 - Update google_fonts from v6.3.1 to v6.3.2 - Update path_provider related packages to latest versions - Update shared_preferences related packages to latest versions
Adds the `logoUrl` field to the `CreateSourceState`. This change updates the state class to manage the new logo URL, including modifications to the constructor, `copyWith` method, `props` for equatability, and the `isFormValid` getter to enforce the new required field.
Introduces the `CreateSourceLogoUrlChanged` event to handle updates to the new `logoUrl` field within the `CreateSourceBloc`.
Updates the `CreateSourceBloc` to fully manage the new `logoUrl` field. This change introduces a new event handler `_onLogoUrlChanged` to update the state with the logo URL from the UI. It also registers this handler and modifies the `_onSavedAsDraft` and `_onPublished` methods to include the `logoUrl` when creating a new `Source` instance, resolving the compilation errors. Comments for future logging have been added to enhance maintainability.
Adds the `logoUrl` field to the `EditSourceState`. This change updates the state class to manage the new logo URL for the editing workflow. It includes modifications to the constructor, `copyWith` method, `props` for equatability, and the `isFormValid` getter to enforce the new required field.
Introduces the `EditSourceLogoUrlChanged` event. This allows the UI to notify the `EditSourceBloc` when the user modifies the `logoUrl` field, enabling state updates for the editing workflow.
Updates the `EditSourceBloc` to fully manage the new `logoUrl` field. This change introduces and registers the `_onLogoUrlChanged` event handler. It also updates the `_onEditSourceLoaded` method to populate the `logoUrl` from the fetched source. Finally, it modifies the `_onSavedAsDraft` and `_onPublished` methods to include the `logoUrl` when updating the `Source` instance. Comments for future logging have been added to enhance maintainability.
Integrates the `logoUrl` input field into the `CreateSourcePage` UI. This change adds the necessary `TextEditingController` and `TextFormField` to allow users to input a URL for the source's logo. The controller is properly initialized and disposed, and the text field is connected to the `CreateSourceBloc` to handle state updates.
Integrates the `logoUrl` input field into the `EditSourcePage` UI. This change adds the necessary `TextEditingController` and `TextFormField` to allow users to edit the URL for the source's logo. The controller is properly initialized, disposed, and updated from the BLoC state. The text field is connected to the `EditSourceBloc` to handle state updates.
Corrects a compilation error in `filter_dialog.dart` by adding the required `logoUrl` parameter to the `Source` constructor within an `orElse` block. This dummy object is used as a fallback and now aligns with the updated `Source` model.
- Add Arabic and English translations for "logoUrl" label - Update descriptions for "saveAdMessage" localization entries
Summary of ChangesHello @fulleni, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request introduces the capability to manage a 'Logo URL' for sources within the content management system. It integrates this new field across the application's BLoC state management, user interface for creation and editing, and provides localization. Additionally, it includes a comprehensive update of various project dependencies to ensure stability and leverage the latest features. Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
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.
Code Review
This pull request introduces a new logoUrl field for sources and updates various dependencies. The changes for the new field are well-integrated across the BLoCs, states, events, and UI. My feedback focuses on improving code quality by removing placeholder log comments, addressing a minor formatting inconsistency, and suggesting a refactoring to reduce significant code duplication in the BLoC files. Overall, the changes are good and align with the PR's objectives.
lib/content_management/bloc/create_source/create_source_bloc.dart
Outdated
Show resolved
Hide resolved
…tion and editing - Remove commented-out log statements in CreateSourceBloc and EditSourceBloc - These comments were likely intended for debugging purposes but are no longer
Status
READY
Description
This pull request introduces the capability to manage a 'Logo URL' for sources within the content management system. It integrates this new field across the application's BLoC state management, user interface for creation and editing, and provides localization. Additionally, it includes a comprehensive update of various project dependencies to ensure stability and leverage the latest features.
Type of Change