Skip to content

refactor: revamp to use data access layer #580

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 147 commits into from
Mar 26, 2023
Merged

refactor: revamp to use data access layer #580

merged 147 commits into from
Mar 26, 2023

Conversation

rakibansary
Copy link
Contributor

@rakibansary rakibansary commented Mar 26, 2023 •
edited
Loading

rakibansary and others added 30 commits October 26, 2022 19:58
@rakibansary
wip
8135ac8
@rakibansary
wip: test xray
f3c8fda
@rakibansary
wip: more tracing
75ef95b
@rakibansary
wip: dont send trace info to bus
7ac1980
@rakibansary
feat: challenge phase extension
3e0a4f6 
* move common phase functionality to a phase helper
* remove post-mortem phase if submission phase is opened
* determine isOpen status based on start and end dates
@rakibansary
fix: incorrect reference
b35ee6f
@rakibansary
fix: actual end date
29ebffc
@rakibansary
fix: actual end time & actual start time
db9bdad
@rakibansary
fix: post-mortem phase
3357873 
* post-mortem phase should be "scheduled" if submission phase is opened
* because of extension
@rakibansary
Merge branch 'feat/tracing' into feat/phase-extension
932cb27
@rakibansary
feat(tracing): include trace info in payload sent to bus
418b797
@rakibansary
fix: only allow Active, Draft challenges to be extended
33e4bb0
@rakibansary
fix: only allow Active, Draft challenges to be extended
69c0f26
@rakibansary
fix (challenge extension): throw error when challenge phase and/or st…
20c4a1d 
…art date modification is not allowed
@rakibansary
fix(challenge update): disallow updating phase/startdate for complete…
78a2587 
…d/cancelled challenges
@rakibansary
refactor: format using prettier
7459c60
@rakibansary
feat: handle grpc errors
6aa8869
@rakibansary
feat: use domain object for challenge timeline template
30814fd
@rakibansary
feat: use domain object for lookups
90730cb
@rakibansary
feat: add support for update operation in challenge timeline template
1968205 
Signed-off-by: Rakib Ansary <[email protected]>
@rakibansary
fix: challenge timeline template delete response
3d7eb5a
@ThomasKranitsas
refactor(domain-challenge): wip timeline template
2298073
@ThomasKranitsas
refactor(domain-challenge): use dal for challenge type db interactions
73e1dc1
@ThomasKranitsas
refactor(domain-challenge): use dal for challenge track, phases, time…
944cfd6 
…line templates
@ThomasKranitsas
refactor(domain-challenge): update attachment and phase helper to use…
caff461 
… dal
@rakibansary
feat: use domain-challenge to create challenge
4f15efb 
Signed-off-by: Rakib Ansary <[email protected]>
@rakibansary
feat: use challenge domain
6591e1f
@rakibansary
chore: lint fixes
8659f02
@ThomasKranitsas
refactor(domain-challenge): integrate update/delete with dal for chal…
d945984 
…lenges
@ThomasKranitsas
refactor(domain-challenge): remove audit logs from challenge service
ae8ef3c
rakibansary and others added 28 commits March 24, 2023 01:16
@rakibansary
Merge branch 'refactor/domain-challenge-dev' into refactor/challenge-…
07c66a0 
…update
@rakibansary
refactor: only update whats necessary in challenge:update
1fd1ea4 
Signed-off-by: Rakib Ansary <[email protected]>
@rakibansary
ci: deploy to dev
396db36 
Signed-off-by: Rakib Ansary <[email protected]>
@rakibansary
fix: missing variables
704f88d 
Signed-off-by: Rakib Ansary <[email protected]>
@rakibansary
fix: enable indexing and resource creation
c44d4d4 
Signed-off-by: Rakib Ansary <[email protected]>
@rakibansary
wip
c1945c3
@rakibansary
feat: remove attributes that match exactly with saved challenge
5e4b07f 
Signed-off-by: Rakib Ansary <[email protected]>
@rakibansary
refactor: challenge update
e70465e
@rakibansary
fix: allow memberId to be a number
8782f0e 
Signed-off-by: Rakib Ansary <[email protected]>
@eisbilir
fix: getM2MToken reference
e3d6dcd
@rakibansary
Merge pull request #572 from topcoder-platform/fix/getm2mtoken
fa398ce 
fix: getM2MToken reference
@rakibansary
fix: winner can not be set for non tasks
669c1b1
@rakibansary
fix: remove hardcoded debug code
243b6e8
@rakibansary
fix: unsetting winners
06578d3 
Signed-off-by: Rakib Ansary <[email protected]>
@eisbilir
fix: phase update object
68a7dfd
@eisbilir
fix: allow private description to be empty
80a4689
@rakibansary
Merge pull request #573 from topcoder-platform/fix/phase-update
99c8645 
fix: phase update object
@eisbilir
fix: phases
f0f4dd7
@eisbilir
Merge pull request #574 from topcoder-platform/fix/phase-update
8375f2f 
fix: phases
@rakibansary
chore: handle phase update
3104444 
Signed-off-by: Rakib Ansary <[email protected]>
@ThomasKranitsas
validation updates
b60daa8
@ThomasKranitsas
unset legacy.directProjectId as we do not allowe this to change
28d1646
@rakibansary
fix: bump up versions
33247ca 
Signed-off-by: Rakib Ansary <[email protected]>
@rakibansary
fix: prize amount in cents
8ce4e02
@rakibansary
Merge pull request #576 from topcoder-platform/fix/update-validation
1c4dd3a 
validation updates
@rakibansary
Merge pull request #578 from topcoder-platform/refactor/challenge-update
ae4c4e7 
fix: challenge update
@rakibansary
Merge pull request #579 from topcoder-platform/refactor/domain-challe…
f28085d 
…nge-dev

fix: challenge update
@rakibansary
fix: search by legacy id
82b5754 
Signed-off-by: Rakib Ansary <[email protected]>
@rakibansary rakibansary changed the title Refactor/domain challenge refactor: revamp to use data access layer Mar 26, 2023
@rakibansary rakibansary merged commit 4b670c1 into dev Mar 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@rakibansary @ThomasKranitsas @jmgasper @eisbilir