Switch useUpdate to useSet

Author: prescottprue

docs/api/firebaseInstance.md

@@ -397,10 +397,11 @@ internally while updating profile on Firestore uses `set` with
 -   `profileUpdate` **[Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object)** Profile data to place in new profile
 -   `options` **[Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object)** Options object (used to change how profile
     update occurs)
-    -   `options.useUpdate` **[Boolean](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Boolean)** Use update instead of set with merge
-        (only used when updating profile on Firestore)
-    -   `options.merge` **[Boolean](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Boolean)** Whether or not to use merge when setting
-        (only used when updating profile on Firestore)
+    -   `options.useSet` **[Boolean](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Boolean)** Use set with merge instead of
+        update. Setting to `false` uses update (can cause issue of profile document
+        does not exist). Note: Only used when updating profile on Firestore (optional, default `true`)
+    -   `options.merge` **[Boolean](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Boolean)** Whether or not to use merge when
+        setting profile. Note: Only used when updating profile on Firestore (optional, default `true`)
 
 Returns **[Promise](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Promise)** 
 

src/createFirebaseInstance.js

@@ -415,10 +415,11 @@ export const createFirebaseInstance = (firebase, configs, dispatch) => {
    * @param {Object} profileUpdate - Profile data to place in new profile
    * @param {Object} options - Options object (used to change how profile
    * update occurs)
-   * @param  {Boolean} options.useUpdate - Use update instead of set with merge
-   * (only used when updating profile on Firestore)
-   * @param  {Boolean} options.merge - Whether or not to use merge when setting
-   * (only used when updating profile on Firestore)
+   * @param  {Boolean} [options.useSet=true] - Use set with merge instead of
+   * update. Setting to `false` uses update (can cause issue of profile document
+   * does not exist). Note: Only used when updating profile on Firestore
+   * @param  {Boolean} [options.merge=true] - Whether or not to use merge when
+   * setting profile. Note: Only used when updating profile on Firestore
    * @return {Promise}
    */
   const updateProfile = (profileUpdate, options) =>

src/utils/auth.js

@@ -197,20 +197,24 @@ export const updateProfileOnRTDB = (firebase, profileUpdate) => {
  * @param  {Object} profileUpdate - Updates to profile object
  * @param  {Object} options - Options object for configuring how profile
  * update occurs
- * @param  {Boolean} options.useUpdate - Use update instead of set with merge
- * @param  {Boolean} options.merge - Whether or not to use merge when setting
- * profile
+ * @param  {Boolean} [options.useSet=true] - Use set with merge instead of
+ * update. Setting to `false` uses update (can cause issue of profile document
+ * does not exist).
+ * @param  {Boolean} [options.merge=true] - Whether or not to use merge when
+ * setting profile
  * @return {Promise} Resolves with results of profile get
  */
 export const updateProfileOnFirestore = (
   firebase,
   profileUpdate,
   options = {}
 ) => {
-  const { useUpdate = false, merge = true } = options
+  const { useSet = true, merge = true } = options
   const { firestore, _: { config, authUid } } = firebase
   const profileRef = firestore().doc(`${config.userProfile}/${authUid}`)
-  const profileUpdatePromise = useUpdate
+  // Use set with merge (to prevent "No document to update") unless otherwise
+  // specificed through options
+  const profileUpdatePromise = useSet
     ? profileRef.set(profileUpdate, { merge })
     : profileRef.update(profileUpdate)
   return profileUpdatePromise.then(() => profileRef.get())