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.

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

Review: 1247-populate-new-default-value #1259

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Review: 1247-populate-new-default-value #1259

wants to merge 1 commit into from

Conversation

aashish-t-magar
Copy link
Collaborator

No description provided.

@aashish-t-magar aashish-t-magar requested a review from sriza June 30, 2023 09:49
@aashish-t-magar aashish-t-magar self-assigned this Jun 30, 2023
@what-the-diff
Copy link

what-the-diff bot commented Jun 30, 2023

PR Summary

  • Addition of New Command SettingPopulateActivityDefaultValue.php
    A new file titled SettingPopulateActivityDefaultValue.php has been incorporated in the app/Console/Commands directory. It houses a newly developed command class named SettingPopulateActivityDefaultValue. This command helps in integrating new keys into the activity_default_values column within the settings table. Essentially, the command fetches chunks of settings—10 at a time—from the database. It then expands the activity_default_values column with new fields, each given an empty value. The updated settings are then saved back to the database.

@aashish-t-magar aashish-t-magar force-pushed the 1247-populate-new-default-value branch from e1b824a to bd069e5 Compare June 30, 2023 09:52
@aashish-t-magar aashish-t-magar force-pushed the 1247-populate-new-default-value branch from bd069e5 to 7a5d6f1 Compare January 16, 2024 08:10
@aashish-t-magar aashish-t-magar force-pushed the 1247-populate-new-default-value branch from 7a5d6f1 to fd69ca0 Compare January 23, 2024 11:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sriza sriza Awaiting requested review from sriza

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

Assignees

@aashish-t-magar aashish-t-magar

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@aashish-t-magar @praweshsth