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

Feat:add a pure and cqrs template for masa framework #11

Merged
merged 5 commits into from
Apr 24, 2023

Conversation

MapleWithoutWords
Copy link
Member

@MapleWithoutWords MapleWithoutWords commented Apr 4, 2023

resolved #27

@doddgu
Copy link
Contributor

doddgu commented Apr 4, 2023

MasaBp-Empty -> BlazorPure, 并且短名为 masab-pure
MasaFx-Service-Cqrs -> FrameworkServiceCqrs,短名为 masafxsp
MasaFx-Service -> FrameworkService,短名为 masafxs

@doddgu doddgu requested a review from zhenlei520 April 4, 2023 05:36
@doddgu
Copy link
Contributor

doddgu commented Apr 4, 2023

@zhenlei520 PTAL

@capdiem
Copy link
Contributor

capdiem commented Apr 6, 2023

@MapleWithoutWords @doddgu

  1. 为什么不拆成三个PR?
  2. 官方BlazorServer Empty项目的短名称是 blazorserver-empty,建议一致。

@capdiem
Copy link
Contributor

capdiem commented Apr 6, 2023

@MapleWithoutWords 建议你在 .NET7 的 BlazorServer Empty 模板上做修改,只需要增加必要的一些代码。

@zhenlei520
Copy link
Contributor

Issues that have been closed need to be rechecked to make sure they have been addressed @MapleWithoutWords

@doddgu
Copy link
Contributor

doddgu commented Apr 10, 2023

@MapleWithoutWords @doddgu

  1. 为什么不拆成三个PR?
  2. 官方BlazorServer Empty项目的短名称是 blazorserver-empty,建议一致。

可以,那就保持一直吧。否则pro和pure都是p开头也容易混淆
@MapleWithoutWords

@MapleWithoutWords
Copy link
Member Author

@MapleWithoutWords @doddgu

  1. 为什么不拆成三个PR?
  2. 官方BlazorServer Empty项目的短名称是 blazorserver-empty,建议一致。

可以,那就保持一直吧。否则pro和pure都是p开头也容易混淆 @MapleWithoutWords

跟官方保持一致,到时候名称会不会冲突,导致不显示?@doddgu @capdiem

@zhenlei520
Copy link
Contributor

跟官方保持一致,到时候名称会不会冲突,导致不显示?@doddgu @capdiem

前面加个masa?

@MapleWithoutWords
Copy link
Member Author

image
我觉得,顺序要跟官方保持一致。官方名称叫:blazorserver-empty。我们就算在前面加masa,那也应该是:masablazor-server-emtpy才对。masablazor-empty-server这个名称感觉按照英文都不顺

@doddgu
Copy link
Contributor

doddgu commented Apr 14, 2023

image 我觉得,顺序要跟官方保持一致。官方名称叫:blazorserver-empty。我们就算在前面加masa,那也应该是:masablazor-server-emtpy才对。masablazor-empty-server这个名称感觉按照英文都不顺

ok

@MapleWithoutWords MapleWithoutWords changed the title Feat/add pure template Feat:add a pure and cqrs template with masa framework Apr 18, 2023
@MapleWithoutWords MapleWithoutWords changed the title Feat:add a pure and cqrs template with masa framework Feat:add a pure and cqrs template for masa framework Apr 18, 2023
@zhenlei520
Copy link
Contributor

MasaBp-Empty -> BlazorPure, 并且短名为 masab-pure MasaFx-Service-Cqrs -> FrameworkServiceCqrs,短名为 masafxsp MasaFx-Service -> FrameworkService,短名为 masafxs

masa framework 缩写为masafx没有问题,但模版名应该能反应其意图,否则这将不利于后期记忆

@zhenlei520
Copy link
Contributor

@MapleWithoutWords 应该有一个简易的图反应每个模版的作用以及对应的技术栈,新建一个issues并关联到当前的pr,确保后续追踪

Copy link
Contributor

@zhenlei520 zhenlei520 left a comment

Choose a reason for hiding this comment

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

检查之前已经被关闭问题,重新再检查一遍

Copy link
Contributor

@zhenlei520 zhenlei520 left a comment

Choose a reason for hiding this comment

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

LGTM @doddgu

@doddgu doddgu merged commit dfc523e into main Apr 24, 2023
@doddgu doddgu deleted the feat/add-pure-template branch April 24, 2023 06:10
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.

feat: Masa Framework new Templates
4 participants