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

feat: update theme to use theming api v4 #499

Merged
merged 232 commits into from
Jul 3, 2024
Merged

Conversation

gosiexon-zen
Copy link
Contributor

@gosiexon-zen gosiexon-zen commented Jul 2, 2024

Description

This PR updates Copenhagen Theme to use Templating API v4 (docs)

Screenshots

Screenshot 2024-07-03 at 10 24 15
Screenshot 2024-07-03 at 10 24 29

Checklist

  • 📗 all commit messages follow the conventional commits standard
  • ⬅️ changes are compatible with RTL direction
  • ♿ Changes to the UI are tested for accessibility and compliant with WCAG 2.1.
  • 📝 changes are tested in Chrome, Firefox, Safari and Edge
  • 📱 changes are responsive and tested in mobile
  • 👍 PR is approved by @zendesk/vikings

anpa and others added 30 commits July 3, 2023 18:05
feat: request form using field data helpers
Fredx87 and others added 17 commits May 22, 2024 13:08
fix: always redirect to the new request page when a ticket form is selected
fix: text reflow a11y issue in the Answer Bot modal
refactor: align with the refactored APIs for the parent request
fix: clearing date field when it is hidden
chore: added test id for suggested articles
docs: added information about the React components
@gosiexon-zen gosiexon-zen requested review from a team as code owners July 2, 2024 14:23
Copy link
Contributor

@anpa anpa left a comment

Choose a reason for hiding this comment

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

🥳 🥳 🥳

@@ -17,6 +25,7 @@ parts:
title: "Title of the footer of the Answer Bot Modal"
screenshot: "https://drive.google.com/file/d/14qsEyUiVTy9MRYBGkgVWLmlduhoscYBw/view?usp=drive_link"
value: "Does this article answer your question?"
obsolete: "2024-08-22"

Choose a reason for hiding this comment

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

Suggested change
obsolete: "2024-08-22"
obsolete: "2024-09-22"

The date should be at least 2 months in the future :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, changed

@kubraokcu kubraokcu added the g11n-commented Commented by Globalization label label Jul 3, 2024
BREAKING CHANGE: theme is now relying on functionality that is exclusive to the theming api v4
@kubraokcu kubraokcu added g11n-approved Commented by Globalization label and removed g11n-commented Commented by Globalization label labels Jul 3, 2024
@gosiexon-zen gosiexon-zen merged commit 611d6cf into master Jul 3, 2024
5 checks passed
@gosiexon-zen gosiexon-zen deleted the mbien/templating-api-v4 branch July 3, 2024 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
g11n-approved Commented by Globalization label
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants