amplication / docs

Amplication docs
https://docs.amplication.com
Apache License 2.0
16 stars 57 forks source link

Docs: .NET Updates #474

Closed dericksozo closed 1 month ago

dericksozo commented 1 month ago

(docs): Making some documentation updates for the new .NET-related changes to Amplication

dericksozo commented 1 month ago

Hey @PazYanoverr, this PR makes changing to the following important pages concerning .NET:

Here are some additional changes we can make:

  1. Update Licensing.md to include information about the differences between the Node.js open-source licensing and the .NET licensing.
  2. Add a new top-level .NET page that explains the key differences and similarities with the Node.js offering.

What are your thoughts?

PazYanoverr commented 1 month ago

@dericksozo - I agree we need to update the license page. Regarding the "new top-level .NET page", I agree it can be beneficial. However, we can start now without it, and explain the key differences and similarities with the Node.js offering- in the specific pages which talk about node.js. The rest of the pages should be shared between the languages (or at least we should aim to it)

dericksozo commented 1 month ago

Hi @PazYanoverr, I've implemented your latest feedback.

PazYanoverr commented 1 month ago

@dericksozo - What about the "licensing" page? Can you please adjust it as well to the new definitions/updates?

dericksozo commented 1 month ago

Definitely, @PazYanoverr. I just implemented a new Licensing page. This one should be much more aligned, but let me know your thoughts!

PazYanoverr commented 1 month ago

@dericksozo - Added comments regarding the licensing page. Thanks!

dericksozo commented 1 month ago

Thanks a lot for the additional feedback, @PazYanoverr and @mulygottlieb. I implement it all in the latest commit.

PazYanoverr commented 1 month ago

@yuval-hazaz - Per @mulygottlieb's suggestion- can you please review the updated licensing page?

yuval-hazaz commented 1 month ago

@yuval-hazaz - Per @mulygottlieb's suggestion- can you please review the updated licensing page?

@PazYanoverr LGTM

dericksozo commented 1 month ago

@PazYanoverr, the latest commit should do it. What are your thoughts?