feat: ✨ add a tally contact form#684
Merged
Merged
Conversation
|
Thank you for submitting this pull request! We appreciate your contribution to the project. Before we can merge it, we need to review the changes you've made to ensure they align with our code standards and meet the requirements of the project. We'll get back to you as soon as we can with feedback. Thanks again! |
Contributor
Reviewer's GuideThis PR embeds a Tally-powered contact form into the Contact Us page by injecting the Tally script via Next.js Head and replacing the custom form component with an iframe; it also tweaks container padding and adds Volta configuration to package.json for consistent Node/Yarn versions. Class diagram for Contact Us page form replacementclassDiagram
class ContactUs {
+render()
}
class ContactForm {
+render()
}
class iframe {
+data-tally-src
+loading
+width
+height
+title
}
ContactUs --|> iframe : uses
ContactUs ..x ContactForm : replaced
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Deploying fairdataihub-org with
|
| Latest commit: |
e1903f9
|
| Status: | 🚫 Build failed. |
|
Thanks for making updates to your pull request. Our team will take a look and provide feedback as soon as possible. Please wait for any GitHub Actions to complete before editing your pull request. If you have any additional questions or concerns, feel free to let us know. Thank you for your contributions! |
Contributor
|
Running Lighthouse audit... |
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Hey @megasanjay - I've reviewed your changes - here's some feedback:
- Consider using Next.js Script component from 'next/script' instead of manually adding a <script> tag in for better loading control and SSR handling.
- The Volta config in package.json is inlined next to the homepage key; please separate and properly format it on its own lines to maintain valid JSON structure.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- Consider using Next.js Script component from 'next/script' instead of manually adding a <script> tag in <Head> for better loading control and SSR handling.
- The Volta config in package.json is inlined next to the homepage key; please separate and properly format it on its own lines to maintain valid JSON structure.
## Individual Comments
### Comment 1
<location> `src/pages/contact-us.tsx:41` </location>
<code_context>
</h2>
- <ContactForm />
+ <iframe
+ data-tally-src="https://tally.so/embed/mRqBLd?alignLeft=1&hideTitle=1&transparentBackground=1&dynamicHeight=1"
+ loading="lazy"
+ width="100%"
+ height="276"
+ title="AI-READI Contact Form"
+ />
</div>
</code_context>
<issue_to_address>
Review iframe accessibility and fallback content.
Consider adding fallback content inside the iframe tag to support users with iframes disabled or screen readers that may not interpret the embedded form correctly.
</issue_to_address>
<suggested_fix>
<<<<<<< SEARCH
<iframe
+ data-tally-src="https://tally.so/embed/mRqBLd?alignLeft=1&hideTitle=1&transparentBackground=1&dynamicHeight=1"
+ loading="lazy"
+ width="100%"
+ height="276"
+ title="AI-READI Contact Form"
+ />
=======
<iframe
data-tally-src="https://tally.so/embed/mRqBLd?alignLeft=1&hideTitle=1&transparentBackground=1&dynamicHeight=1"
loading="lazy"
width="100%"
height="276"
title="AI-READI Contact Form"
aria-label="Contact form for feedback or collaboration"
>
Your browser does not support iframes, or iframes are disabled.
Please visit <a href="https://tally.so/r/mRqBLd" target="_blank" rel="noopener noreferrer">this link</a> to access the contact form.
</iframe>
>>>>>>> REPLACE
</suggested_fix>Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
Member
Author
📝 Changed routes:
1 deleted route:
Commit e1903f9 (https://fairdataihub-website-fjp1u98cb-fairdataihub.vercel.app). |
Co-authored-by: sourcery-ai[bot] <58596630+sourcery-ai[bot]@users.noreply.github.com>
|
Thanks for closing this pull request! If you have any further questions, please feel free to open a new issue. We are always happy to help! |
github-actions Bot
pushed a commit
that referenced
this pull request
Jul 18, 2025
# [2.52.0](v2.51.0...v2.52.0) (2025-07-18) ### Features * ✨ add a tally contact form ([#684](#684)) ([848d2f8](848d2f8))
Member
|
🎉 This PR is included in version 2.52.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary by Sourcery
Integrate Tally contact form into the Contact Us page and enforce Node/Yarn versions via Volta
New Features:
Build: