Merge branch 'master' into feature/extract-console-tab

.github/workflows/build.yml

@@ -78,6 +78,10 @@ jobs:
           cache-dependency-path: |
             openreview-api/package-lock.json
             openreview-web/package-lock.json
+      - name: Set up Python 3.10
+        uses: actions/setup-python@v5
+        with:
+          python-version: '3.10'
       - name: Setup openreview-py
         run: |
           cd openreview-py

components/profile/EmailsSection.js

@@ -68,6 +68,7 @@ const EmailsSection = ({
   updateEmails,
   institutionDomains,
   isNewProfile,
+  loadProfile,
 }) => {
   const router = useRouter()
 
@@ -78,7 +79,7 @@ const EmailsSection = ({
         let emailCopy = { ...email }
         if (email.key === action.data.key) {
           emailCopy = action.data
-          if (action.setVerifyVisible) emailCopy.verifyVisible = action.data.visibleValue
+          emailCopy.verifyVisible = false
         }
         return emailCopy
       })
@@ -98,7 +99,7 @@ const EmailsSection = ({
         const emailCopy = { ...email }
         if (email.key === action.data.key) {
           emailCopy.confirmed = true
-          if (action.setVerifyVisible) emailCopy.verifyVisible = action.data.visibleValue
+          emailCopy.verifyVisible = false
         }
         return emailCopy
       })
@@ -120,7 +121,7 @@ const EmailsSection = ({
         return emailCopy
       })
     }
-
+    if (action.reset) return action.data
     return state
   }
   // eslint-disable-next-line max-len
@@ -134,8 +135,7 @@ const EmailsSection = ({
   const handleAddEmail = () => {
     setEmails({
       addNewEmail: true,
-      setVerifyVisible: true,
-      data: { email: '', key: nanoid(), isValid: true, visibleValue: false },
+      data: { email: '', key: nanoid(), isValid: true },
     })
   }
 
@@ -145,16 +145,14 @@ const EmailsSection = ({
     const isValid = isValidEmail(targetValue.toLowerCase())
     setEmails({
       updateEmail: true,
-      setVerifyVisible: true,
-      data: { ...existingEmailObj, key, email: targetValue, isValid, visibleValue: false },
+      data: { ...existingEmailObj, key, email: targetValue, isValid },
     })
   }
 
   const handleRemoveEmail = (key) => {
     setEmails({
       removeEmail: true,
-      setVerifyVisible: true,
-      data: { key, visibleValue: false },
+      data: { key },
     })
   }
 
@@ -192,16 +190,25 @@ const EmailsSection = ({
     const newEmail = emails?.find((p) => p.key === key)?.email?.toLowerCase()
     const token = emails?.find((p) => p.key === key)?.verificationToken ?? ''
     const payload = { email: newEmail, token }
+    let verifyResult
     try {
       if (isNewProfile) {
         await api.put(`/activatelink/${router.query.token}`, payload, { accessToken })
       } else {
-        await api.put('/activatelink', payload, { accessToken })
+        verifyResult = await api.put('/activatelink', payload, { accessToken })
+      }
+      if (verifyResult?.id) {
+        const updatedProfile = await loadProfile()
+        setEmails({
+          reset: true,
+          data:
+            updatedProfile?.emails?.map((p) => ({ ...p, key: nanoid(), isValid: true })) ?? [],
+        })
+        return promptMessage(`${newEmail} has been verified`)
       }
       setEmails({
-        setVerifyVisible: true,
         setConfirmed: true,
-        data: { key, visibleValue: false },
+        data: { key },
       })
       if (isInstitutionEmail(newEmail, institutionDomains)) setHasInstitutionEmail(true)
       return promptMessage(`${newEmail} has been verified`)

components/profile/ProfileEditor.js

@@ -30,6 +30,7 @@ export default function ProfileEditor({
   loading,
   isNewProfile,
   saveProfileErrors,
+  loadProfile,
 }) {
   const profileReducer = (state, action) => {
     if (action.type === 'reset') return action.data
@@ -487,6 +488,7 @@ export default function ProfileEditor({
               updateEmails={(emails) => setProfile({ type: 'emails', data: emails })}
               institutionDomains={institutionDomains}
               isNewProfile={isNewProfile}
+              loadProfile={loadProfile}
             />
           </ProfileSection>
         )

components/profile/ProfileSection.js

@@ -2,7 +2,7 @@ export default function ProfileSection({ title, instructions, children }) {
   return (
     <section>
       <h4>{title}</h4>
-      <p className="instructions">{instructions}</p>
+      <div className="instructions">{instructions}</div>
 
       {children}
     </section>

package.json

@@ -1,6 +1,6 @@
 {
   "name": "openreview-web",
-  "version": "1.13.6",
+  "version": "1.13.8",
   "private": true,
   "scripts": {
     "dev": "next dev --port $NEXT_PORT",

pages/about.js

@@ -17,6 +17,12 @@ const About = () => (
 
     <div className="row">
       <div className="col-xs-12 col-md-10 col-md-offset-1">
+        <p>
+          OpenReview project is now a 501(c)(3) organization under the name OpenReview Foundation.
+        </p>
+        <p>
+          For inquiries, please contact <a href="mailto:[email protected]">[email protected]</a> or <a href="mailto:[email protected]">[email protected]</a>.
+        </p>
         <p>
           OpenReview aims to promote openness in scientific communication, particularly the
           peer review process, by providing a flexible cloud-based web interface and underlying

pages/profile/edit.js

@@ -28,12 +28,13 @@ export default function ProfileEdit({ appContext }) {
       if (profiles?.length > 0) {
         const formattedProfile = formatProfileData(profiles[0], true)
         setProfile(formattedProfile)
-      } else {
-        setError({ statusCode: 404, message: 'Profile not found' })
+        return formattedProfile
       }
+      setError({ statusCode: 404, message: 'Profile not found' })
     } catch (apiError) {
       setError({ statusCode: apiError.status, message: apiError.message })
     }
+    return null
   }
 
   const unlinkPublication = async (profileId, noteId) => {
@@ -178,6 +179,7 @@ export default function ProfileEdit({ appContext }) {
         cancelHandler={() => router.push('/profile').then(() => window.scrollTo(0, 0))}
         loading={loading}
         saveProfileErrors={saveProfileErrors}
+        loadProfile={loadProfile}
       />
     </div>
   )

styles/pages.scss

@@ -21,4 +21,3 @@
 @import 'pages/tasks';
 @import 'pages/venue';
 @import 'pages/venues';
-@import 'pages/confirm';