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

Update for newest API #122

Merged
merged 6 commits into from
Jan 15, 2025
Merged

Update for newest API #122

merged 6 commits into from
Jan 15, 2025

Conversation

Jym77
Copy link
Contributor

@Jym77 Jym77 commented Jan 14, 2025

Resolves II-8839, resolves II-8376

  • Update the upload URL to the new one, this won't work until it the new endpoint is alive.
  • Switch siteID from a string to a number.
  • Stop sending individual rules durations.
  • Update yarn.

@Jym77 Jym77 self-assigned this Jan 14, 2025
Copy link

changeset-bot bot commented Jan 14, 2025

🦋 Changeset detected

Latest commit: 16affd3

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 27 packages
Name Type
@siteimprove/alfa-test-utils Minor
@siteimprove/alfa-angular Minor
@siteimprove/alfa-assert Minor
@siteimprove/alfa-chai Minor
@siteimprove/alfa-cheerio Minor
@siteimprove/alfa-cli Minor
@siteimprove/alfa-command Minor
@siteimprove/alfa-crawler Minor
@siteimprove/alfa-cypress Minor
@siteimprove/alfa-enzyme Minor
@siteimprove/alfa-formatter-earl Minor
@siteimprove/alfa-formatter-json Minor
@siteimprove/alfa-formatter-sarif Minor
@siteimprove/alfa-formatter Minor
@siteimprove/alfa-frontier Minor
@siteimprove/alfa-interviewer Minor
@siteimprove/alfa-jasmine Minor
@siteimprove/alfa-jest Minor
@siteimprove/alfa-jquery Minor
@siteimprove/alfa-playwright Minor
@siteimprove/alfa-puppeteer Minor
@siteimprove/alfa-react Minor
@siteimprove/alfa-scraper Minor
@siteimprove/alfa-selenium Minor
@siteimprove/alfa-unexpected Minor
@siteimprove/alfa-vue Minor
@siteimprove/alfa-webdriver Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@Jym77 Jym77 added the major Backwards-incompatible change that touches public API label Jan 15, 2025
@Jym77 Jym77 marked this pull request as ready for review January 15, 2025 14:36
@Jym77 Jym77 requested a review from a team as a code owner January 15, 2025 14:36
Copy link
Contributor

@rcj-siteimprove rcj-siteimprove left a comment

Choose a reason for hiding this comment

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

Looks good 👍🏼

Copy link
Contributor

@jfer-siteimprove jfer-siteimprove left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@Jym77 Jym77 merged commit 5404311 into main Jan 15, 2025
4 checks passed
@Jym77 Jym77 deleted the update-api branch January 15, 2025 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
major Backwards-incompatible change that touches public API
Projects
Status: 🏁 Done
Development

Successfully merging this pull request may close these issues.

3 participants