Skip to content

Fix form issues, update sign-up button, and add zoom feature#143

Open
Pramad712 wants to merge 7 commits intomainfrom
bug-fixes
Open

Fix form issues, update sign-up button, and add zoom feature#143
Pramad712 wants to merge 7 commits intomainfrom
bug-fixes

Conversation

@Pramad712
Copy link
Copy Markdown
Contributor

@Pramad712 Pramad712 commented Apr 12, 2026

  1. Fix linking in markdown
  2. Fix time slot js
  3. Make sign up button grey when no more spots available
  4. Use embed form if and only if a form url is given in for the event. Otherwise, use closest match of custom forms.
  5. Add zoom feature for posters pop-up

Note: The jsconfig was also edited since the TS errors make vscode go crazy about issues across the codebase

Copilot AI review requested due to automatic review settings April 12, 2026 04:51
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 12, 2026

🚀 Expo preview is ready!

  • Project → audacity-music-club
  • Platforms → android, ios
  • Runtime Version → 1.0.0
  • More info

Learn more about 𝝠 Expo Github Action

@Pramad712 Pramad712 requested a review from Heliophist April 12, 2026 04:54
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the volunteer opportunity flow by improving link handling in Markdown descriptions, adjusting sign-up button behavior when events are full, refining time-slot selection behavior, and adding zoom support to the posters/programs gallery.

Changes:

  • Add explicit Markdown link handling and visually disable the Sign Up button when full/already submitted.
  • Update form-selection logic and time-slot range picker behavior.
  • Add zoomable posters gallery support (new dependency + gesture handler root wrapper).

Reviewed changes

Copilot reviewed 9 out of 10 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
src/screens/VolunteerOpportunityScreen.js Adds Markdown onLinkPress handling; styles Sign Up button as disabled when full/submitted.
src/screens/VolunteerFormScreen.js Changes fuzzy matching behavior for selecting a custom form; removes explicit unknown-form error path.
src/components/TimeSlot.js Adjusts range-selection picker behavior to reopen for end-time selection.
src/components/TextField.js Tweaks input height.
src/components/PostersButton.js Adds zoom support to poster modal/gallery; optimizes FlatList rendering.
src/components/NextButton.js Adds disabled styling support.
App.js Wraps app in GestureHandlerRootView for gesture/zoom support.
package.json Adds react-native-reanimated-zoom dependency.
package-lock.json Locks react-native-reanimated-zoom dependency.
jsconfig.json Adjusts JS type-checking compiler options.
Comments suppressed due to low confidence (1)

src/screens/VolunteerOpportunityScreen.js:108

  • onLinkPress calls Linking.openURL(url) without handling rejection, which can produce unhandled promise rejections for unsupported/malformed URLs. Prefer using the existing openURL helper from src/utils (it already catches and shows a user-facing Alert) or add a .catch(...) here.
              </Pressable>
            </View>
          </View>

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/screens/VolunteerFormScreen.js
Comment thread src/screens/VolunteerFormScreen.js
Comment thread jsconfig.json
Heliophist
Heliophist previously approved these changes Apr 16, 2026
Copy link
Copy Markdown
Contributor

@Heliophist Heliophist left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

k

Copy link
Copy Markdown
Contributor

@Heliophist Heliophist left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants