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

Development comparisons #271

Merged
merged 15 commits into from
Oct 17, 2024
Merged

Development comparisons #271

merged 15 commits into from
Oct 17, 2024

Conversation

jhadobe
Copy link
Contributor

@jhadobe jhadobe commented Sep 13, 2024

This PR adds comparison info between in-process and out-of-process extensibility. Highlighting what is available in Commerce vs what is achievable with App Builder + Commerce.

staging:

@jhadobe jhadobe self-assigned this Sep 13, 2024
@jhadobe jhadobe added the new-topic A major update published as an entirely new document label Sep 13, 2024
@jhadobe
Copy link
Contributor Author

jhadobe commented Sep 13, 2024

A few questions for @keharper:

  • Do you have ideas for better page names?
  • Should I break the third-party integrations section into a separate page?
  • app-development-comparison looks a little flat due to the similarity between H2/H3, but the table structure was not conducive either. Any ideas on visually separating these sections? All I can think of is to use more unordered lists, like in services-comparison.md.
  • How do we want to credit the developer?

* Database data
* Filesystem

### Frontend development
Copy link

Choose a reason for hiding this comment

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

how about to add here a note that native HTML + CSS is also an option?

@jhadobe jhadobe requested a review from nuzil September 24, 2024 15:30
@nuzil
Copy link

nuzil commented Sep 25, 2024

Will pass it also to @keharper
We discussed that we need to add 2 sections for

  1. Instead of treating in-process and out-of-process are equals, we should be clear that unless absolutely needed, we recommend developers always choose out of process
  2. In services comparison, we should be clear that modifying the default MySQL database can cause upgrade issues, so we recommend using App Builder with state database for simple app specific data, and a 3rd party database for more complex needs.

My suggestion is to add block for point 1:

In-process extensibility model currently has a lot of disaadvantages like, increasing maintenance costs, reducing software stability, increasing AC server overload and more.
Looking on those factors and also on increasign of business compexity and technologies stack, we recommends to use out-of-process model as much as it possible

And similar for 2nd

@jhadobe
Copy link
Contributor Author

jhadobe commented Sep 25, 2024

Will pass it also to @keharper We discussed that we need to add 2 sections for

  1. Instead of treating in-process and out-of-process are equals, we should be clear that unless absolutely needed, we recommend developers always choose out of process
  2. In services comparison, we should be clear that modifying the default MySQL database can cause upgrade issues, so we recommend using App Builder with state database for simple app specific data, and a 3rd party database for more complex needs.

My suggestion is to add block for point 1:

In-process extensibility model currently has a lot of disaadvantages like, increasing maintenance costs, reducing software stability, increasing AC server overload and more.
Looking on those factors and also on increasign of business compexity and technologies stack, we recommends to use out-of-process model as much as it possible

And similar for 2nd

Thanks @nuzil I had already added bolded "Recommended method" text to all the "App Builder" sections to help clearly emphasize that OOP is the best option. I've also added a version of your quote above.

Copy link
Contributor

Choose a reason for hiding this comment

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

"Each approach has its own set of advantages and use cases, and understanding them is crucial for developers and businesses looking to maximize their platform's potential" - Can we change this line to say something like "Adobe recommends adopting out-of-process extensibility where possible due to its ease of maintenance."

jhadobe and others added 2 commits October 4, 2024 10:43
Co-authored-by: slamech <111585301+slamech@users.noreply.github.com>
@jhadobe jhadobe requested a review from slamech October 4, 2024 15:45
keharper and others added 2 commits October 17, 2024 13:00
Co-authored-by: slamech <111585301+slamech@users.noreply.github.com>
@keharper keharper merged commit 64c0761 into main Oct 17, 2024
5 checks passed
@keharper keharper deleted the jh_comparison branch October 17, 2024 18:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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