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

Database storage #52

Merged
merged 7 commits into from
Jul 27, 2023
Merged

Database storage #52

merged 7 commits into from
Jul 27, 2023

Conversation

keharper
Copy link
Contributor

Purpose of this pull request

This pull request (PR) adds the DB storage topic to the ASC best practices

Affected pages

  • ...

Links to Magento Open Source code

  • ...

@keharper keharper added internal Differentiates work between external and internal contributors new-topic A major update published as an entirely new document labels Jul 21, 2023
@keharper keharper requested a review from jhadobe July 21, 2023 21:46
@keharper keharper changed the title Asc database Database storage Jul 24, 2023
@naomirobichaud
Copy link

Looks great to me - nice job restructuring the benefits/limitations sections, very clear now

@keharper keharper merged commit 376ddd9 into main Jul 27, 2023
2 checks passed
@keharper keharper deleted the ASC-database branch July 27, 2023 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
internal Differentiates work between external and internal contributors new-topic A major update published as an entirely new document
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants