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

Updated composer depedencies to be the same as the advanced application #262

Merged
merged 31 commits into from
Jun 17, 2022
Merged

Updated composer depedencies to be the same as the advanced application #262

merged 31 commits into from
Jun 17, 2022

Conversation

developedsoftware
Copy link
Contributor

@developedsoftware developedsoftware commented Jun 6, 2022 •
edited
Loading

Q A
Is bugfix? no
New feature? no
Breaks BC? no
Tests pass? yes
Fixed issues comma-separated list of tickets # fixed by the PR, if any

Closes #263

Luke English and others added 25 commits February 23, 2022 17:58
Compatible with PHP 8.1 and Codeception 5
f92f294
Compatible with PHP 8.1 and Codeception 5
f0d1c23
@samdark
Add PHP 8.1 to workflow
ac9bbec
@developedsoftware
30c8260
@developedsoftware
Merge origin/master
ec20b93
@developedsoftware
php 7.4 as a minimum
162b67d
@developedsoftware
fixed broken tests
3cf78c5
@developedsoftware
fixed broken tests
7958e02
@developedsoftware
fixed broken tests
a6d44a3
@developedsoftware
fixed broken tests
29d1de1
@developedsoftware
Merge branch 'yiisoft:master' into master
50723d7
@developedsoftware
className() is deprecated
13c528f
@developedsoftware
Merge branch 'yiisoft:master' into master
3d034d9
@developedsoftware
updated dependencies
dbd7933
@developedsoftware
Merge origin/master
b988c28
@developedsoftware
Merge branch 'yiisoft:master' into master
2134adf
@developedsoftware
Updated composer depedencies to work with Codeception 5
0c48fb0
@developedsoftware
removed composer.lock
f9da2d5
@developedsoftware
5043916
@developedsoftware
downgraded to bootstrap4 as its not relevant to this PR
7b284ab
@developedsoftware
output path is not defined by key "paths: output"
0c8c6ad
@developedsoftware
downgraded to swiftmailer as its not relevant to this PR
0701819
@developedsoftware
update to bootstrap 5
22f0bd6
@developedsoftware
replace class_name with actor in codecept
030e10d
@developedsoftware
use symfonymailer
1c90420
@developedsoftware developedsoftware changed the title Updated composer depedencies to work with Codeception 5 Updated composer depedencies to be the same as the advanced application Jun 6, 2022
@developedsoftware
Copy link
Contributor Author

developedsoftware commented Jun 6, 2022 •
edited
Loading

Hmm not sure if this is an issue with GitHub CI but I am getting references to swiftmailer even though in test.php I have and tests pass locally...

'mailer' => [
            'class' => 'yii\symfonymailer\Mailer',
            // send all mails to a file by default. You have to set
            // 'useFileTransport' to false and configure transport
            // for the mailer to send real emails.
            'useFileTransport' => true,
        ],

@developedsoftware developedsoftware mentioned this pull request Jun 6, 2022
Closed
@developedsoftware
Copy link
Contributor Author

Not sure how I can get this work so will have to park it for now. I have no reference to swiftmailer in the code so it must be pulling in that code from somewhere else as part of the CI @samdark

@developedsoftware
Copy link
Contributor Author

Will require #264 to be merged in before checks will pass.

@samdark samdark merged commit 9d3b845 into yiisoft:master Jun 17, 2022
@samdark
Copy link
Member

samdark commented Jun 17, 2022

Thanks!

@samdark samdark added this to the 2.0.46 milestone Jun 17, 2022
@WinterSilence WinterSilence mentioned this pull request Jul 6, 2022
Open
@particleflux particleflux mentioned this pull request Oct 6, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
2.0.46
Development

Successfully merging this pull request may close these issues.

Should this be in sync with the advanced template?
2 participants
@developedsoftware @samdark