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

Adapt to last changes in Yii DataView and Yii Form #605

Merged
merged 2 commits into from
Dec 2, 2023
Merged

Conversation

vjik
Copy link
Member

@vjik vjik commented Nov 27, 2023

Q A
Is bugfix? ✔️
New feature?
Breaks BC?
Fixed issues -

Copy link

what-the-diff bot commented Nov 27, 2023

PR Summary

  • Addition of 'yiisoft/data' Dependency
    The project's dependency list was updated to include yiisoft/data. This new software package is now an integral part of our project, and the version we are using is the development version, dubbed as version 1.0.

  • Updates in user/index.php

    • Usage of User Class: The standard class for managing users has been incorporated in our project.
    • Column Creation Method Change: The previous method for creating columns has been replaced with a new method, making our code cleaner and efficient.
    • Refactoring Callback Functions: The functions that are called back for processing content have been renamed with clear type hinting, which means they are now easier to understand.
    • Column Header Display: The way we construct columns has been updated to include a header argument, which will help us in displaying information in structured, descriptive columns.

@vjik vjik added the status:code review The pull request needs review. label Nov 27, 2023
@vjik vjik requested a review from a team November 27, 2023 12:37
@vjik vjik changed the title Adapt to last changes in Yii DataView Adapt to last changes in Yii DataView and Yii Form Dec 2, 2023
@vjik vjik merged commit 0331470 into master Dec 2, 2023
15 checks passed
@vjik vjik deleted the adapt-dataview branch December 2, 2023 12:04
rossaddison added a commit to rossaddison/invoice that referenced this pull request Dec 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status:code review The pull request needs review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants