Skip to content

Update README.md with latest RFC#14

Open
Laurgrin wants to merge 1 commit intopaysera:masterfrom
Laurgrin:patch-1
Open

Update README.md with latest RFC#14
Laurgrin wants to merge 1 commit intopaysera:masterfrom
Laurgrin:patch-1

Conversation

@Laurgrin
Copy link

No description provided.

fields. Even if there is one (besides `id` and `updatedAt`) field in entity, extra one can be added later and this
would break the functionality (or naming).

### Directory stucture
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should go under Symfony related conventions section, not "entities"


### Directory stucture

For new projects using Symfony > 4.0, we use flat directory structure, and do not use bundles. If we need a bundle, we should reconsider our project's needs and structure - maybe this bundle would make more sense as a library or another service. Needless to say, this does not apply to existing projects, so if a project already uses bundles, we continue that. **Why?** Because this encourages smaller, more self-contained projects.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lets add small example with suggested paths for microservice which deals with more than one domain, which is not uncommon.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants