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

Hotfix/change alias name #16

Open
wants to merge 65 commits into
base: main
Choose a base branch
from

Conversation

shunnyjang
Copy link
Member

No description provided.

shunnyjang and others added 30 commits October 7, 2022 17:55
wrong mappedBy property occurred error so that delete it
fix sql bug that all case when condition's result was TRUE
modify prize_code property type in Reward because prize_code property in Prize that referenced column's type is string
Change checking whether a user has a reward and paying reward point from when retrieve user owned rewards to each time when user register answer, user completed challenge, or user make new relation. Because Tranditional method occurred transaction, so that I changed like this way
Added missing ID and date search criteria that should be mandatory
Merge pull request #35 from shunnyjang/feat/fix-reward-bug
Modifying from Array Type to Object Type at Client Developer's Request
fix bug occurred when user send RSVP to other user and create new notification. related user nickname is not property physically saved on db so add persist disabled in property option
* feat: Init Push Notification Schedule

* feat: create initial schedule when signed up

* fix: transaction bug

duplicate transaction occurred bug so that remove from creating reward
* fix: friend community criteria

add criteria that only confirmed friends' answers are displayed

* fix: typo

fix swagger typo that sql query result doesn't recognize camel upper case

* fix: modify nickname eager option abled

To include related user's nickname in notification, modify related user nickname's eager option abled

* feat: send notification to friends

feature that sending notification about user register answer of challenge or user complete challenge bucket to friends who have a relationship with user

* feat: send notfication to friends

* feat: add order criteria

* fix: transaction bug
* modify: delete unused import

* update: mailer package

* feat: set mailer config

* feat: email module

* feat: send register email, activate user api

* update: dependency packages

* feat: register auth code entity

* feat: change register email to send auth code

- change template to send auth code to user
- modify mail template rendering method
- add email controller

* feat: generate auth code and send mail when register

* feat: move signup email service on auth module

* fix: logo image url

* feat: modify to unify singup and send verifying email

* fix: wrong property name

* update: swagger docs
* hotfix: change reard logic async

sync logic occured server down so change logic to async and make nothing's happen when logic occurred transaction bug

* feat: add logger
1) get status that notification exists which is unread
2) update all unread notifcations to be read
1) add is_deleted property in answer
2) add criteria in query answer
3) api method to initialize bucket
shunnyjang and others added 30 commits October 18, 2022 15:26
1. add condition that filter buckets' status not abandon
2. use with query to retrieve user's friend list whom visibility status is not private
3. inner join friends view in friends' answers list
to know how many users' bucket is going on with this challenge, add bucket count on challenge detail api
- change app name to block from report: to include broad sense of blocking
- blookup blocked list api, check is target user blocked api
- modify report func return value to always return sucess even target user had blocked by user
* feat: fix indent style and delete unused files
* feat: add info whether user owned challenge
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.

1 participant