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(email): handling email case sensitive #165

Merged
merged 3 commits into from
Aug 1, 2024
Merged

Conversation

idilhaq
Copy link

@idilhaq idilhaq commented Jul 23, 2024

No description provided.

internal/store/postgres/appeal_repository.go Outdated Show resolved Hide resolved
internal/store/postgres/appeal_repository.go Outdated Show resolved Hide resolved
core/appeal/service.go Show resolved Hide resolved
core/appeal/service.go Show resolved Hide resolved
@rahmatrhd rahmatrhd merged commit e89f850 into main Aug 1, 2024
9 checks passed
@rahmatrhd rahmatrhd deleted the email_case_insensitive branch August 1, 2024 04:02
harishkswami pushed a commit to harishkswami/guardian that referenced this pull request Aug 23, 2024
* feat(email): handling email case sensitive

* chore: handling list page

* chore: handle case on filter

---------

Co-authored-by: Muhammad Idil Haq Amir <idil.amir@tokopedia.com>
rahmatrhd added a commit that referenced this pull request Aug 27, 2024
* lark support

* lark support

* lark support

* lark support

* lark support

* lark support

* lark support

* lark support

* lark support

* lark support

* lark support

* lark support

* lark support

* lark support

* lark support

* lark support

* lark support

* lark support

* lark support

* lark support

* fix: fix typo in mapstructure annotation

* fix: map old notifier config to new multiple notifiers format

* fix: fix slack client initialization with multiple workspaces

* fix: fix lint errors

* add json config for notifiers

* add json config for notifiers

* add json config for notifiers

* add json config for notifiers

* add json config for notifiers

* add json config for notifiers

* fix review commits

* fix review commits

* feat(email): handling email case sensitive (#165)

* feat(email): handling email case sensitive

* chore: handling list page

* chore: handle case on filter

---------

Co-authored-by: Muhammad Idil Haq Amir <idil.amir@tokopedia.com>

* feat(gate): introduce gate provider (#164)

* feat(gate): introduce gate provider

* fix: send gate api key in query param

* fix: handle non-success http codes

* feat(notifier): expose variables in notifier criteria (#166)

* feat: add support for google_oauth2 token. Related to issue: https://… (#169)

* feat: add support for google_oauth2 token. Related to issue: #168

* add test cases

* add test cases

* add error formatting changes

---------

Co-authored-by: anjali.agarwal <anjali.aggarwal@gojek.com>

* feat: add support for expression in request body when fetching metada… (#170)

* feat: add support for expression in request body when fetching metadata sources for appeal

* add a condition in http mock call

---------

Co-authored-by: anjali.agarwal <anjali.aggarwal@gojek.com>

* ci: lock golangci-lint to v1.59 (#172)

* ci: lock golangci-lint to v1.59

* ci: fix golangci-lint warnings

* chore: fetch creator details before fetching metadata sources as they… (#174)

chore: fetch creator details before fetching metadata sources as they are helpful

Co-authored-by: anjali.agarwal <anjali.aggarwal@gojek.com>

---------

Co-authored-by: Rahmat Hidayat <rahmatramahidayat@gmail.com>
Co-authored-by: Muhammad Idil Haq Amir <idil.amir@gotocompany.com>
Co-authored-by: Muhammad Idil Haq Amir <idil.amir@tokopedia.com>
Co-authored-by: anjali9791 <anjalia271@gmail.com>
Co-authored-by: anjali.agarwal <anjali.aggarwal@gojek.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