Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Sep 16, 2025

The "Log in" link and "Sign up" button in the mobile navigation menu were not clickable, making it impossible for mobile users to access authentication flows.

Root Cause
The issue was caused by undefined environment variables in the client-side React component. The variables SIGN_UP_URL and LOG_IN_URL were not available in the browser context, resulting in href attributes with undefined values that rendered the links non-functional.

Solution
Replaced environment variable references with hard-coded URLs for the authentication links:

// Before (non-functional)
const signUpUrl = import.meta.env.SIGN_UP_URL;
const logInUrl = import.meta.env.LOG_IN_URL;

// After (functional)
const signUpUrl = "https://app.operately.com/sign_up";
const logInUrl = "https://app.operately.com/log_in";

This approach ensures reliable functionality across all environments while maintaining the same URLs that were already defined in the environment variables.

Testing

  • ✅ Mobile "Log in" link now properly navigates to authentication page
  • ✅ Mobile "Sign up" button now properly navigates to registration page
  • ✅ Desktop navigation continues to work correctly
  • ✅ No regression in other navigation functionality

Before/After Screenshots

Before fix: "Log in" appears as plain text, "Sign up" button is styled but non-functional
Before fix

After fix: Both "Log in" and "Sign up" are fully functional clickable links
After fix

Fixes #218.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • telemetry.astro.build
    • Triggering command: node /home/REDACTED/work/website/website/node_modules/.bin/astro dev --port 4321 (dns block)

If you need me to access, download, or install something from one of these locations, you can either:


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copy link
Contributor Author

Copilot AI commented Sep 16, 2025

@markoa 👋 This repository doesn't have Copilot instructions. With Copilot instructions, I can understand the repository better, work faster and produce higher quality PRs.

I can generate a .github/copilot-instructions.md file for you automatically. Click here to open a pre-filled issue and assign it to me. I'll write the instructions, and then tag you for review.

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Sep 16, 2025

Deploying operately-website with  Cloudflare Pages  Cloudflare Pages

Latest commit: 445e8dd
Status: ✅  Deploy successful!
Preview URL: https://1da8877e.operately-website.pages.dev
Branch Preview URL: https://copilot-fix-218.operately-website.pages.dev

View logs

Co-authored-by: markoa <8651+markoa@users.noreply.github.com>
Copilot AI changed the title [WIP] “Log in” link and “Sign up” button are not tappable on mobile Fix "Log in" and "Sign up" buttons not tappable in mobile navigation Sep 16, 2025
Copilot AI requested a review from markoa September 16, 2025 21:35
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.

“Log in” link and “Sign up” button are not tappable on mobile

2 participants