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

Tet 740 omis market #6821

Draft
wants to merge 12 commits into
base: migration-deploy
Choose a base branch
from
Draft

Tet 740 omis market #6821

wants to merge 12 commits into from

Conversation

bau123
Copy link
Contributor

@bau123 bau123 commented May 21, 2024

Fix omis market call

Document what the PR does and why the change is needed

Test instructions

What should I see?

Screenshots

Before

Add a screenshot

After

Add a screenshot

Checklist

  • Has the branch been rebased to main?
  • Automated tests (Any of the following when applicable: Unit, Functional or End-to-End)
  • Manual compatibility testing (Browsers: Chrome, Firefox, Edge, Safari)

marijnkampf and others added 12 commits May 17, 2024 13:51
* Upgrade postgres, open search and redis

* Rename /healthcheck end point to /pingdom
* Add ASIM format to logger in production. Tag logs as express startup/request.
* Add default parameter to end point for metadata requests to fix Hawk issue.
* Use urls.metadata for investment profile filter options.
* Exclude end point from a11y check
* Refactor CollectionItem to allow tags to be rendered in the same way as in activity cards

* Refactor events collection page to use the API instead of ActivityStream

* Update tests

* Cleanup dead code
@bau123 bau123 changed the base branch from main to migration-deploy May 21, 2024 09:19
Copy link

cypress bot commented May 21, 2024

Passing run #53467 ↗︎

0 15 0 0 Flakiness 0

Details:

Update OmisMarkets.js
Project: data-hub-frontend Commit: 26cbbfb681
Status: Passed Duration: 02:30 💡
Started: May 21, 2024 9:25 AM Ended: May 21, 2024 9:27 AM

Review all test suite changes for PR #6821 ↗︎

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.

None yet

4 participants