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.

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

🔖 Rc/1.9.0 #468

Merged
merged 53 commits into from
Nov 28, 2024
Merged

🔖 Rc/1.9.0 #468

merged 53 commits into from
Nov 28, 2024

Conversation

anncatton
Copy link
Contributor

anncatton and others added 30 commits October 17, 2023 17:42
@anncatton
Merge pull request #446 from icgc-argo/main
2e4eb0e 
Main mergeback
@yalturmes
Update Jenkinsfile
d9041c2
@joneubank
Merge pull request #448 from icgc-argo/jenkins-migration
242c95d 
Update Jenkinsfile
@anncatton
Update docker-compose, .env.example files. add axios dep
49b0512
@anncatton
Update config, secrets values. Create egaClient with token fetch
4227b2b
@anncatton
initial refresh token logic
0e06d57
@anncatton
add ega endpoint constants
cfa6d35
@anncatton
add section for env vars in readme
ce6d8a7
@anncatton
remove logs
912e16a
@anncatton
remove public key reference
83be068
@anncatton
Add zod, update ts. Add expiry to approved app list data
aa42318
@anncatton
Add api response types, permissions and users get funcs
870ba74
@anncatton
Add safe parsing for api list results. Add dacId to config
6162104
@anncatton
add tsdocs
3570e4a
@anncatton
Add safeParse checks to ega client calls.
76a740e
@anncatton
Add failure returns to ega client calls, restructure types
f62d34b
@anncatton
remove empty file
14d35f4
@anncatton
move failure types to separate file, add appId to approved users data
12bb890
@anncatton
modify list response types
e995c79
@anncatton
Create ega permissions main job function, add happy path flow
965f0bb
@anncatton
move safeParseArray to own file, update imports
e866b2b
@anncatton
Merge pull request #458 from icgc-argo/453-ega-api-funcs
ab2e9bc 
453 ega api funcs
@anncatton
Merge branch '452-ega-auth-client' into 454-implement-job-flow
cacf7cb
@anncatton
WIP - fix for pagination when checking permissions
ef48443
@anncatton
Expand tsdocs
04879b4
@anncatton
add comment for api bug, reorg ega services
951bebc
@anncatton
move custom errors to types dir
f4be3e8
@anncatton
➕ Add pThrottle dependency as source code, rate limit ega client funcs (
816df19 
#461)

* add pThrottle source code dir, add throttling to all egaClient functions
@anncatton
rename types + util functions for clarity
9476767
@anncatton
remove unneeded fields from EgaPermissionRequest response schema
81d23b6
@anncatton
Add refresh token logic for 401 errors. Fetch all user permissions in…
5bbda86 
… one req
@anncatton
Merge pull request #460 from icgc-argo/454-implement-job-flow
db89e47 
✨ 454 - implement ega job flow
@anncatton
Add ega public key fetch to secrets
e92dab7
@anncatton
Split clients into separate files. Fix error handling order
b25cca9
@anncatton
Adds ega job report data. Adds job to main batch job, with feature fl…
9cb3949 
…ag enabling
@anncatton
add newline, cleanup
041962a
@anncatton
add axios-retry to egaClient
d7e93d1
@anncatton
improve ega report structure, add example report to tsdoc
4d3a9d1
@anncatton
remove unnecessary comments
7880d28
@anncatton
reject promise when max retry limit exceeded, add retry limits to con…
5a1d79b 
…fig, readme
@anncatton
Merge pull request #463 from icgc-argo/455-implement-error-retry
afbb2c0 
🥅 455 - Error handling and retries + job reporting
@anncatton
Merge pull request #462 from icgc-argo/add-refresh-token-flow
a227b1e 
✨ 454 - Add refresh token logic to Axios client
@anncatton
Merge pull request #457 from icgc-argo/452-ega-auth-client
f23c7dc 
✨🔧 452 - [Feature branch] - Add ega permissions integration
@anncatton
update build script to match new project structure
fe79fd3
@anncatton
Merge pull request #464 from icgc-argo/ega-test-build
84f3eb3 
🔨 🚚 Update build script to match new project structure
@anncatton
update dockerfile cmd to use correct dir
7aaa960
@anncatton
Merge pull request #465 from icgc-argo/update-dockerfile-cmd
d6b7aff 
update dockerfile cmd to use correct dir
@anncatton
import fetch for node 16
c0fd907
@anncatton
Merge pull request #466 from icgc-argo/fix-missing-fetch-def
fd3bc06 
import fetch for node 16
@anncatton
remove .length check to ensure permissions are added if no existing r…
aaa0ff8 
…esults
@anncatton
remove todo
07bd5c9
@anncatton
Merge pull request #467 from icgc-argo/fix-adding-permissions-when-em…
6d3e208 
…pty-result

Fixes failure to add permissions when none exist
@anncatton
🔖 rc 1.9.0
0bd4d5d
@anncatton anncatton merged commit 091049f into main Nov 28, 2024
2 checks passed
@anncatton anncatton deleted the rc/1.9.0 branch November 28, 2024 22:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@joneubank joneubank joneubank approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@anncatton @joneubank @yalturmes