Skip to content

Extract feedback form URL to environment configuration#340

Merged
iimpulse merged 2 commits intoenhancement/standalone-optimizefrom
copilot/sub-pr-329
Feb 11, 2026
Merged

Extract feedback form URL to environment configuration#340
iimpulse merged 2 commits intoenhancement/standalone-optimizefrom
copilot/sub-pr-329

Conversation

Copy link
Contributor

Copilot AI commented Feb 11, 2026

Moves hardcoded Google Forms URL from FeedbackComponent to environment configuration files for proper environment-specific configuration management.

Changes:

  • Added HPO_FEEDBACK_FORM_URL constant to environment.ts, environment.dev.ts, and environment.prod.ts
  • Updated FeedbackComponent to import and reference environment.HPO_FEEDBACK_FORM_URL

Before:

export class FeedbackComponent {
  private formUrl = 'https://docs.google.com/forms/d/e/1FAIpQLSfID5F6is5J7GSJUlFzIYZ7FOsXEebjLjLZwXhLdFPELVBVMQ/viewform?usp=dialog';
  
  constructor(private sanitizer: DomSanitizer) {
    this.safeFormUrl = this.sanitizer.bypassSecurityTrustResourceUrl(this.formUrl);
  }
}

After:

import { environment } from '../../../environments/environment';

export class FeedbackComponent {
  constructor(private sanitizer: DomSanitizer) {
    this.safeFormUrl = this.sanitizer.bypassSecurityTrustResourceUrl(environment.HPO_FEEDBACK_FORM_URL);
  }
}

All environments currently use the same form URL since feedback is collected to a single destination.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: iimpulse <7397665+iimpulse@users.noreply.github.com>
Copilot AI changed the title [WIP] Add changes to address feedback on optimization enhancement Extract feedback form URL to environment configuration Feb 11, 2026
Copilot AI requested a review from iimpulse February 11, 2026 19:23
@iimpulse iimpulse marked this pull request as ready for review February 11, 2026 19:38
@iimpulse iimpulse merged commit 730a6c6 into enhancement/standalone-optimize Feb 11, 2026
1 check passed
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.

2 participants

Comments