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

[Chore] Merge develop into sep-6 #1186

Merged
merged 4 commits into from
Nov 3, 2023

Conversation

philipliu
Copy link
Contributor

Description

Merges develop into sep-6.

Context

Keep sep-6 branch up to date.

Testing

  • ./gradlew test

Documentation

N/A

Known limitations

N/A

Ifropc and others added 4 commits November 1, 2023 13:08
…lback (stellar#1183)

### Description

Fix account not being forwarded in SEP-9's /customer callback

### Context

When receiving callback for SEP-9 field forwarding all JWT fields
(account and memo) was missing. Turned out, they were not being
forwarded properly

### Testing

- `./gradlew test`
-  Adjusted test scenario 

### Documentation

There is a task for documenting this feature 

### Known limitations

N/A
…r#1180)

### Description

- Split SEP Tests of AnchorPlatformIntegrationTest
- Replace `SepClient` classes with wallet sdk for some of the SEP tests.
- Disable the tests that requires non-`default` test configuration.
- Enable integration test concurrency.

### Context

- Test improvements.

### Testing

- `./gradlew test`

### Known Limitaions
- Tests with non-`default` configurations will be moved to
`extended-tests` subproject.
@philipliu philipliu changed the base branch from develop to sep-6 November 2, 2023 22:38
@philipliu philipliu marked this pull request as ready for review November 2, 2023 22:38
Copy link
Collaborator

@lijamie98 lijamie98 left a comment

Choose a reason for hiding this comment

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

LGTM

@philipliu philipliu merged commit 7e2f8f9 into stellar:sep-6 Nov 3, 2023
5 checks passed
@philipliu philipliu deleted the sep-6-final-merge branch November 3, 2023 14:02
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.

3 participants