-
Notifications
You must be signed in to change notification settings - Fork 47
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
4.4.2 Release #446
Closed
4.4.2 Release #446
+32,366
−78,647
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
new request header
dynamically import signals sdk in start of protect method export fr-auth callbacks of protect in index update changelog
fix(javascript-sdk): fix-exports-update-protect
feat: new request header
update the module type in package.json
fix(ping-protect): update-module-type
remove circular-dep-on-config
fix(javascript-sdk): circular-dep
update signals sdk to latest
fix(javascript-sdk): change x-requested-platform to opt-in
fix(ping-protect): update-signals-sdk
the old esm module resolution would be fine if we wrote imports with .js at the end. we could use 'bundler' but then 'bundler' has odd support of module types in package.json Further node16 supports all of the above.
build(ping-protect): update-module-resolution
update bundle to leverage vite for now. then generate types with tsc.
fix(ping-protect): bundling
update extensions in imports in token-vault and ping-protect. leverage node16 and es2020 in tsconfig
refactor: update-nx-e2e-tests-builds
update signals sdk to 5.3.0
fix(ping-protect): update-ping-protect-signals-sdk
we need the idToken for pingone signout and we also remove it in case we are not in a pingone signout. so we will store the idToken and pass it in, incase. default to empty string
redirect test case in autoscript
fix(javascript-sdk): add logout redirect for pingone
update to use vite because of extensions needed in esm
update to use esmodules for js files
build(javascript-sdk): update-build-to-vite-lib
update versions in package.json
chore(javascript-sdk): package-versions
update changelogs
chore(javascript-sdk): changelogs
☁️ Nx Cloud ReportCI is running for commit d3d4e55. 📂 Click to track the progress, see the status, the terminal output, and the build insights. Sent with 💌 from NxCloud. |
This is an initial commit of the mock api saving work as progress is made
Initial iteration of the mock-api using effect and effect-http I included a readme with a lot of context as to how to add code and where to look.
…ilities feat(javascript-sdk): refactor authorize URL utilities for DaVinci
refactored the api to clean up the fetch service. renamed from repository to services, and abstracted the test code into a new file. Created a map of responses to infer types based on the url passed in.
Refactoring how services are being used in mock api to make it more | maintainble and easier to follow. The goal should be readability and make it so its clearer and easier to narrow responses
update the types to be a bit better across the app. created a types file for centralizing query and header types for generics
Layers are the convention that should be used when needing to provide requirements to effects
renamed files and imports for style requests
implemented basic tests for some services in effect
update test file and some cleanup of code
refactor to update effect-http version which added a few methods and improved http-error. refactor arrow functions to function named style
added more tests to cover some of the internal functions
use opaque types for schemas to make reading types cleaner. refactor non-effect returning tests to use `it` instead of `it.effect`
autoscript-apps will now wait for the new mock api as well to start. so its usable in the existing test suite
Mock api v2
update nx to 19.5.1
chore: update-nx
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
JIRA Ticket
Please link jira ticket here
Description
Type of Change
Please Delete options that are not relevant
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Definition of Done
Check all that apply
Documentation