yashodgayashan / environment_platform

A comprehensive online request management platform for the Ministry of Environment and Wildlife Resources, Sri Lanka
Apache License 2.0
6 stars 4 forks source link

Development #9

Closed yashodgayashan closed 4 years ago

deshankoswatte commented 4 years ago

@yashodgayashan, @sanjulamadurapperuma, @vikumkbv

sanjulamadurapperuma commented 4 years ago
  • For the folder names shall we have core and dashboard instead of backend and frontend?
  • Also the ballerina module should be named to env_platform instead of envPlatform I think.

@yashodgayashan, @sanjulamadurapperuma, @vikumkbv

+1 for changing the folder names to core and dashboard.

as for the ballerina module name, is there a reason behind this?

deshankoswatte commented 4 years ago

as for the ballerina module name, is there a reason behind this?

No specific reason but it looks more readable have to wait and see @yashodgayashan's response

yashodgayashan commented 4 years ago
  • For the folder names shall we have core and dashboard instead of backend and frontend?
  • Also the ballerina module should be named to env_platform instead of envPlatform I think.

@yashodgayashan, @sanjulamadurapperuma, @vikumkbv

It is good to me.

sanjulamadurapperuma commented 4 years ago

as for the ballerina module name, is there a reason behind this?

No specific reason but it looks more readable have to wait and see @yashodgayashan's response

In that case, we have to switch module naming conventions to underscores instead of camel case moving on in the development. If that's okay then +1 from me.

yashodgayashan commented 4 years ago

I will regenerate and do it give me 1 hr.