Skip to content

feat: more info reminder config#619

Merged
sergiojoker11 merged 5 commits into
mainfrom
feat/more-info-reminder-config
May 15, 2025
Merged

feat: more info reminder config#619
sergiojoker11 merged 5 commits into
mainfrom
feat/more-info-reminder-config

Conversation

@sergiojoker11
Copy link
Copy Markdown
Member

@sergiojoker11 sergiojoker11 commented May 14, 2025

depends on : Notifycal/shared#136

extended ReminderConfig with:

  • language
  • timezone
  • T&C, privacy policy and marketing consent
  • company sector and size

Comment thread package.json Outdated
Comment thread src/utils/datetime.ts Outdated
Comment thread src/utils/datetime.ts Outdated
Comment thread src/lambdas/api/patch-user-profile/index.ts Outdated
dsiguero
dsiguero previously approved these changes May 14, 2025
Copy link
Copy Markdown
Member

@dsiguero dsiguero left a comment

Choose a reason for hiding this comment

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

1 comment about potentially leveraging zod's input and output schemas for "mutating" data from the schema

@sergiojoker11 sergiojoker11 force-pushed the feat/more-info-reminder-config branch from 23abf1c to 99ce1d0 Compare May 15, 2025 12:16
@sergiojoker11 sergiojoker11 marked this pull request as ready for review May 15, 2025 12:27
@sergiojoker11 sergiojoker11 merged commit cb3b996 into main May 15, 2025
4 checks passed
@sergiojoker11 sergiojoker11 deleted the feat/more-info-reminder-config branch May 15, 2025 12:35
sergiojoker11 pushed a commit that referenced this pull request May 15, 2025
🤖 I have created a release *beep* *boop*
---


##
[0.32.0](v0.31.0...v0.32.0)
(2025-05-15)


### Features

* more info reminder config
([#619](#619))
([cb3b996](cb3b996))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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