Skip to content

[ConfigBuilder] Adding PHP ConfigBuilders for DoctrineBundle #15273

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

Merged
merged 1 commit into from
Apr 22, 2021

Conversation

Nyholm
Copy link
Member

@Nyholm Nyholm commented Apr 21, 2021

All doctrine bundle configuration is converted

@Nyholm Nyholm changed the title Adding PHP ConfigBuilders for DoctrineBundle [Doctrine][ConfigBuilder] Adding PHP ConfigBuilders for DoctrineBundle Apr 21, 2021
@carsonbot carsonbot changed the title [Doctrine][ConfigBuilder] Adding PHP ConfigBuilders for DoctrineBundle [ConfigBuilder] Adding PHP ConfigBuilders for DoctrineBundle Apr 22, 2021
@javiereguiluz javiereguiluz force-pushed the doctrine-config-builder branch from cd65603 to 14ea9c7 Compare April 22, 2021 07:34
@javiereguiluz
Copy link
Member

Nice! Thank you

@javiereguiluz javiereguiluz merged commit 5c37580 into symfony:5.x Apr 22, 2021
@Nyholm
Copy link
Member Author

Nyholm commented Apr 22, 2021

Thank you for merging

@Nyholm Nyholm deleted the doctrine-config-builder branch April 22, 2021 07:36
@Kaaly
Copy link

Kaaly commented Jan 26, 2023

Hi,

i tried to use example for DoctrineConfig like this :

        // config/packages/doctrine.php
        use Symfony\Config\DoctrineConfig;

        return static function (DoctrineConfig $doctrine) {
            $dbalDefault = $doctrine->dbal()
                ->connection('default');
            $dbalDefault->mappingType('enum', 'string');
        };

But no way to find Symfony\Config\DoctrineConfig.

Did i miss anything ?

@Nyholm
Copy link
Member Author

Nyholm commented Jan 26, 2023

Hey

it should be in your cache folder. The files are generated from your installed bundles.

@Kaaly
Copy link

Kaaly commented Jan 27, 2023

ok found it !

Thanks !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants