Skip to content

Commit 56dd060

Browse files
Merge pull request #716 from OpenSignLabs/fix_guestlogin
2 parents f8145fe + 960fbdb commit 56dd060

File tree

3 files changed

+28
-21
lines changed

3 files changed

+28
-21
lines changed

apps/OpenSign/src/components/AddSigner.js

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -49,20 +49,19 @@ const AddSigner = (props) => {
4949
e.preventDefault();
5050
e.stopPropagation();
5151
setIsLoader(true);
52+
if (localStorage.getItem("TenantId")) {
5253
try {
5354
const contactQuery = new Parse.Object("contracts_Contactbook");
5455
contactQuery.set("Name", name);
5556
contactQuery.set("Phone", phone);
5657
contactQuery.set("Email", email);
5758
contactQuery.set("UserRole", "contracts_Guest");
5859

59-
if (localStorage.getItem("TenantId")) {
6060
contactQuery.set("TenantId", {
6161
__type: "Pointer",
6262
className: "partners_Tenant",
6363
objectId: localStorage.getItem("TenantId")
6464
});
65-
}
6665

6766
try {
6867
const _users = Parse.Object.extend("User");
@@ -188,6 +187,9 @@ const AddSigner = (props) => {
188187
setIsLoader(false);
189188
alert("something went wrong!");
190189
}
190+
}else{
191+
alert('something went wrong, please try again later')
192+
}
191193
};
192194

193195
// Define a function to handle the "add yourself" checkbox

apps/OpenSign/src/layout/HomeLayout.js

Lines changed: 23 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -28,29 +28,34 @@ const HomeLayout = () => {
2828
const [tourConfigs, setTourConfigs] = useState([]);
2929
const [, setCookie] = useCookies(["accesstoken", "main_Domain"]);
3030

31+
const tenantId = localStorage.getItem("TenantId");
3132
useEffect(() => {
32-
(async () => {
33-
try {
34-
// Use the session token to validate the user
35-
const userQuery = new Parse.Query(Parse.User);
36-
const user = await userQuery.get(Parse.User.current().id, {
37-
sessionToken: localStorage.getItem("accesstoken")
38-
});
39-
if (user) {
40-
localStorage.setItem("profileImg", user.get("ProfilePic") || "");
41-
checkIsSubscribed();
42-
} else {
33+
if (!tenantId) {
34+
setIsUserValid(false);
35+
} else {
36+
(async () => {
37+
try {
38+
// Use the session token to validate the user
39+
const userQuery = new Parse.Query(Parse.User);
40+
const user = await userQuery.get(Parse.User.current().id, {
41+
sessionToken: localStorage.getItem("accesstoken")
42+
});
43+
if (user) {
44+
localStorage.setItem("profileImg", user.get("ProfilePic") || "");
45+
checkIsSubscribed();
46+
} else {
47+
setIsUserValid(false);
48+
}
49+
} catch (error) {
50+
// Session token is invalid or there was an error
4351
setIsUserValid(false);
4452
}
45-
} catch (error) {
46-
// Session token is invalid or there was an error
47-
setIsUserValid(false);
48-
}
49-
})();
50-
saveCookies();
53+
})();
54+
saveCookies();
55+
}
5156

5257
// eslint-disable-next-line react-hooks/exhaustive-deps
53-
}, []);
58+
}, [tenantId]);
5459
//function to use save data in cookies storage
5560
const saveCookies = () => {
5661
const main_Domain = window.location.origin;

apps/OpenSign/src/pages/Dashboard.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ const Dashboard = () => {
2424
getDashboard(localStorage.getItem("PageLanding"));
2525
}
2626
} else {
27-
navigate("/", { replace: true, state: { from: location } });
27+
navigate("/", { replace: true, state: { from: "" } });
2828
}
2929
// eslint-disable-next-line
3030
}, [id]);

0 commit comments

Comments
 (0)