homeflowjs 0.10.12 → 0.10.13

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "homeflowjs",
3
- "version": "0.10.12",
3
+ "version": "0.10.13",
4
4
  "description": "JavaScript toolkit for Homeflow themes",
5
5
  "main": "index.js",
6
6
  "scripts": {
@@ -21,22 +21,45 @@ const MarketingPreferencesForm = ({ buttonClass, buttonSpanClass, strict }) => {
21
21
  const handleSubmit = (e) => {
22
22
  e.preventDefault();
23
23
 
24
- if (!user.marketing_preference_id) {
24
+ if (!user?.user_id) {
25
25
  notify('Please sign in to update your preferences', 'error');
26
26
  return;
27
27
  }
28
28
 
29
- fetch(`/marketing_preferences/${user.marketing_preference_id}`, {
29
+ /**
30
+ * cms created users don't have marketing_preference_id
31
+ * associated to them. To update marketing preferences
32
+ * we can use any of those 2 endpoints depending on the
33
+ * data we pass. Ideally this should get fixed by BE at some point.
34
+ */
35
+ const url = user?.marketing_preference_id
36
+ ? `/marketing_preferences/${user.marketing_preference_id}`
37
+ : `/custom_marketing_preferences/${user.user_id}`;
38
+
39
+ const optInAtAndUrl = {
40
+ opt_in_marketing_at: optInMarketing ? 'optin' : 'optout',
41
+ opt_in_marketing_url: document.URL,
42
+ };
43
+
44
+ const body = {
45
+ user_id: user.user_id,
46
+ ...(!user.marketing_preference_id ? ({
47
+ marketing_preferences: {
48
+ is_opted_in_to_marketing: optInMarketing,
49
+ ...optInAtAndUrl,
50
+ },
51
+ }) : ({
52
+ ...optInAtAndUrl,
53
+ })),
54
+ };
55
+
56
+ fetch(url, {
30
57
  method: 'POST',
31
58
  headers: {
32
59
  'Content-Type': 'application/json',
33
60
  'X-Http-Method-Override': 'put',
34
61
  },
35
- body: JSON.stringify({
36
- opt_in_marketing_at: optInMarketing ? 'optin' : 'optout',
37
- opt_in_marketing_url: document.URL,
38
- user_id: user.user_id,
39
- }),
62
+ body: JSON.stringify(body),
40
63
  }).then((res) => {
41
64
  if (res.ok && (res.status === 200 || res.status === 204)) {
42
65
  dispatch(setCurrentUser({