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

SL-18629 AIS Fetch #121

Merged
merged 4 commits into from
Mar 15, 2023
Merged

SL-18629 AIS Fetch #121

merged 4 commits into from
Mar 15, 2023

Conversation

akleshchev
Copy link
Contributor

@akleshchev akleshchev commented Mar 14, 2023

This isn't full support yet, doesn't handle a lot of cases (ex: 503), but appears to be good enough to make thumbnails work and allow spliting further work if needed.

P.S. next on the list for fetching: 503 handling (over limit), better load order, may be a way to filter out dupplicate requests.

@github-actions
Copy link

CLA Assistant Lite bot:
Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


1 out of 2 committers have signed the CLA.
@akleshchev
@maxim-productengine
You can retrigger this bot by commenting recheck in this Pull Request

@github-actions github-actions bot added the c/cpp label Mar 14, 2023
@akleshchev akleshchev merged commit 7a70d93 into DRTVWR-567 Mar 15, 2023
@akleshchev akleshchev deleted the andreyk/SL-18629 branch March 15, 2023 18:18
@github-actions github-actions bot locked and limited conversation to collaborators Mar 15, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants