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

Uxp v7.4 #108

Merged
merged 9 commits into from
Apr 12, 2024
Merged

Uxp v7.4 #108

merged 9 commits into from
Apr 12, 2024

Conversation

padmkris123
Copy link
Collaborator

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

| [capture] | `boolean` | `false` |
| eventName | `*` | |
| callback | `*` | |
| options | `boolean` \| `Object` | Boolean value denoting capture value or options object. Currently supports only capture in options object ({ capture: bool_value }). |
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bhatiaPankaj @kareemmyk Can you confirm if these changes are intentional? The change to eventTarget also impacts a lot of HTMLElements.

@@ -25,7 +25,7 @@ Generates cryptographically strong random values

| Param | Type | Description |
| --- | --- | --- |
| array | `*` | An integer-based TypedArray, that is one of Int8Array, Uint8Array, Uint8ClampedArray, Int16Array, Uint16Array, Int32Array, Uint32Array, BigInt64Array, BigUint64Array but not Float32Array nor Float64Array |
| array | `IntegerArray` | An integer-based TypedArray, that is one of Int8Array, Uint8Array, Uint8ClampedArray, Int16Array, Uint16Array, Int32Array, Uint32Array, BigInt64Array, BigUint64Array but not Float32Array nor Float64Array |
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kareemmyk @bhatiaPankaj Can you confirm if this change is required?


| Param | Type | Description |
| --- | --- | --- |
| init | `string` \| `Headers` | Object containing any HTTP headers you want to pre-populate your Headers with. |
| [init] | `Headers` \| `Array<{string, string}>` \| `Object` | An existing Headers object, an array of name-value pairs or object literal with string values. |
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kareemmyk @bhatiaPankaj Can you confirm if this change is required?

@@ -17,7 +18,7 @@ Request class represents a resource request.
| Param | Type | Description |
| --- | --- | --- |
| input | `string` \| `Request` | Defines the resource that you wish to fetch. This can either be: A string containing the URL of the resource you want to fetch. A Request object. |
| options | `Object` | 'Optional' object containing any custom settings that you want to apply to the request. |
| options | `Object` | '(Optional)' object containing any custom settings that you want to apply to the request. |
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kareemmyk @bhatiaPankaj Can you confirm if this change is required?

@padmkris123 padmkris123 merged commit 45f5845 into main Apr 12, 2024
1 check passed
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