synced/Checkmate
1
0
Fork 0
mirror of https://github.com/bluewave-labs/Checkmate.git synced 2025-03-31 22:10:17 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #897 from bluewave-labs/fix/settings

Browse Source 
removed console logs, removed permission check for getting settings
This commit is contained in:
Alexander Holliday
2024-09-30 00:21:17 -07:00
committed by GitHub
parent b99cdaca4e 31509247de
commit 34389d4878
3 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
Client/src/Pages/AdvancedSettings/index.jsx
View File

@ -47,7 +47,6 @@ const AdvancedSettings = ({ isAdmin }) => {
const getSettings = async () => {
const action = await dispatch(getAppSettings({ authToken }));
if (action.payload.success) {
console.log(action.payload.data);
setLocalSettings(action.payload.data);
} else {
createToast({ body: "Failed to get settings" });

4
Client/src/Utils/NetworkService.js
View File

@ -10,7 +10,9 @@ class NetworkService {
this.setBaseUrl(baseURL);
this.unsubscribe = store.subscribe(() => {
const state = store.getState();
baseURL = state.settings.apiBaseUrl || BASE_URL;
if (state.settings.apiBaseUrl) {
baseURL = state.settings.apiBaseUrl;
}
this.setBaseUrl(baseURL);
});
this.axiosInstance.interceptors.response.use(

2
Server/routes/settingsRoute.js
View File

@ -3,7 +3,7 @@ const settingsController = require("../controllers/settingsController");
const { isAllowed } = require("../middleware/isAllowed");
const Monitor = require("../models/Monitor");
router.get("/", isAllowed(["superadmin"]), settingsController.getAppSettings);
router.get("/", settingsController.getAppSettings);
router.put(
"/",
isAllowed(["superadmin"]),