Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix feedback form #490

Merged
merged 5 commits into from
May 30, 2024
Merged

Fix feedback form #490

merged 5 commits into from
May 30, 2024

Conversation

mahmoudadly-cobalt
Copy link
Contributor

@mahmoudadly-cobalt mahmoudadly-cobalt commented May 29, 2024

Ticket: https://zombie.atlassian.net/browse/FX-4175

Changelog

Updated

Fixed the overwrite partial created initially in #101 by

  • adding proper anchor selectors
  • moving the script as is, which was not loading, into the same partial to avoid touching the theme
  • and fixing some minimal stuff related to how an anchor is disabled and changing from GA events to GTM events

Note: I could only validate that the event is pushed to the dataLayer. But I can't validate the data in Google Tag Manager before merging.

(the feedback form style hasn't changed 👇 )
Screenshot 2024-05-29 at 15 42 04

(a thank you not appears for a short time 👇 )
Screenshot 2024-05-29 at 15 43 17

(the buttons stay disabled afterwards 👇 )
Screenshot 2024-05-29 at 15 42 17

(the event appears in the dataLayer object 👇 )
Screenshot 2024-05-29 at 15 49 15

Preview This Change

To see how this change looks in production, scroll down to Deploy Preview. Select the link that looks like https://deploy-preview-<num>--cobalt-docs.netlify.app/

Variables

Help us support a “Write once, publish everywhere” single source of truth. If you see a line that looks like:

{{% asset-categories %}}

You’ve found a shortcode that we include in multiple documents.

You’ll find the content of the shortcode in the following directory:

https://github.com/cobalthq/cobalt-product-public-docs/tree/main/layouts/shortcodes

That shortcode has the same base name as what you see in the PR, such as asset-categories.html.

Checklist for PR Author

[ ] Did you check for broken links and alt text?

Be sure to check for broken links and Alt Text issues. We have a partially automated process,
as described in this section of our repository README:
Test Links and Alt Attributes.

Copy link

netlify bot commented May 29, 2024

Deploy Preview for cobalt-docs ready!

Name Link
🔨 Latest commit 586e9ab
🔍 Latest deploy log https://app.netlify.com/sites/cobalt-docs/deploys/66582a887ed49e0008b42c58
😎 Deploy Preview https://deploy-preview-490--cobalt-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@mahmoudadly-cobalt mahmoudadly-cobalt force-pushed the fix/FX-4175/fix-feedback-form-2 branch from f966e0b to db7d00a Compare May 29, 2024 11:13
@mahmoudadly-cobalt mahmoudadly-cobalt marked this pull request as ready for review May 29, 2024 13:50
@mahmoudadly-cobalt mahmoudadly-cobalt requested a review from a team May 29, 2024 13:50
};

dataLayer.push(args);
Copy link
Contributor

Choose a reason for hiding this comment

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

Where does dataLayer come from? What is it?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@mahmoudadly-cobalt mahmoudadly-cobalt requested a review from a team May 30, 2024 07:29
@mahmoudadly-cobalt mahmoudadly-cobalt merged commit a5afefc into main May 30, 2024
12 checks passed
@mahmoudadly-cobalt mahmoudadly-cobalt deleted the fix/FX-4175/fix-feedback-form-2 branch May 30, 2024 07:31
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.

3 participants