Skip to content

Commit

Permalink
Merge pull request #233 from webreinvent/2.x-feature/session-fixes
Browse files Browse the repository at this point in the history
2.x feature 2.x develop | Updated: session keys in env
  • Loading branch information
themodernpk authored Jan 19, 2024
2 parents 8f903c9 + 35af503 commit e9dd125
Show file tree
Hide file tree
Showing 6 changed files with 246 additions and 24 deletions.
18 changes: 1 addition & 17 deletions Http/Controllers/SetupController.php
Original file line number Diff line number Diff line change
Expand Up @@ -299,7 +299,7 @@ public function testConfigurations(Request $request)

$rules = array(
'app_env' => 'required',
'app_name' => 'required',
'app_name' => 'required|alpha_dash',
'app_timezone' => 'required',
'mail_from_name' => 'required',
'mail_from_address' => 'required',
Expand All @@ -326,24 +326,8 @@ public function testConfigurations(Request $request)
return response()->json($response);
}



//verify mail configuration if set
/*if($request->has('mail_is_valid') && $request->mail_is_valid == false)
{
if($request->has('mail_provider') && !empty($request->mail_provider))
{
$response['success'] = false;
$response['errors'][] = 'Test the mail configuration';
return response()->json($response);
}
}*/


$response = VaahSetup::verifyAppUrl($request);



if(!$response['success'])
{
return response()->json($response);
Expand Down
231 changes: 231 additions & 0 deletions Resources/assets/backend/vaahtwo/build/main.js

Large diffs are not rendered by default.

2 changes: 2 additions & 0 deletions Resources/views/templates/env.blade.php
Original file line number Diff line number Diff line change
Expand Up @@ -28,8 +28,10 @@

BROADCAST_DRIVER=log
CACHE_DRIVER=file
CACHE_PREFIX={{$data->app_name ? strtolower($data->app_name): '' }}
QUEUE_CONNECTION=sync
SESSION_DRIVER=file
SESSION_COOKIE={{$data->app_name ? strtolower($data->app_name): '' }}
SESSION_LIFETIME=120

REDIS_HOST=127.0.0.1
Expand Down
1 change: 0 additions & 1 deletion Vue/vaahtwo/pages/public/setup/intall/Account.vue
Original file line number Diff line number Diff line change
Expand Up @@ -130,7 +130,6 @@ onMounted(async () => {
:suggestions="store.filtered_country_codes"
:completeOnFocus="store.autocomplete_on_focus"
@complete="store.searchCountryCode"
@blur="store.setFocusDropDownToTrue"
@item-select="store.onSelectCountryCode"
placeholder="Enter Your Country"
optionLabel="name"
Expand Down
4 changes: 3 additions & 1 deletion Vue/vaahtwo/pages/public/setup/intall/Configuration.vue
Original file line number Diff line number Diff line change
Expand Up @@ -93,11 +93,13 @@ onMounted(async () => {
<h5 class="text-left p-1 title is-6">App/Website Name</h5>
<div class="p-input">
<InputText v-model="store.config.env.app_name"
placeholder="App/Website Name"
placeholder="Enter your website or app name"
name="config-app_name"
class="p-inputtext-sm" id="app-name"
data-testid="configuration-app_name"
required
@update:modelValue="store.onUpdateAppName"
@keydown.space.prevent
/>
<div class="required-field hidden"></div>
</div>
Expand Down
14 changes: 9 additions & 5 deletions Vue/vaahtwo/stores/setup.js
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ export const useSetupStore = defineStore({

},
env:{
app_name: "VaahCMS",
app_name: null,
app_key: null,
app_debug: 'true',
app_env: null,
Expand Down Expand Up @@ -622,7 +622,7 @@ export const useSetupStore = defineStore({

},
env:{
app_name: "VaahCMS",
app_name: null,
app_key: null,
app_debug: 'true',
app_env: null,
Expand Down Expand Up @@ -657,12 +657,12 @@ export const useSetupStore = defineStore({

setTimeout(() => {
if (!event.query.trim().length) {
this.filtered_country_codes = this.assets.country_calling_codes;
this.filtered_country_codes = vaah().clone(this.assets.country_calling_codes);
}
else {
this.filtered_country_codes = this.assets.country_calling_codes.filter((country) => {
this.filtered_country_codes = vaah().clone(this.assets.country_calling_codes.filter((country) => {
return country.name.toLowerCase().startsWith(event.query.toLowerCase());
});
}));
}
}, 250);
},
Expand Down Expand Up @@ -690,6 +690,10 @@ export const useSetupStore = defineStore({
this.config.count_installed_progress = 100;
},
//---------------------------------------------------------------------
onUpdateAppName: function (value) {
this.config.env.app_name = value.replace(/\s/g,'');
},
//---------------------------------------------------------------------

//---------------------------------------------------------------------
async installDependencies() {
Expand Down

0 comments on commit e9dd125

Please sign in to comment.