Skip to content

Conversation

@olegStrapier
Copy link
Contributor

What does it do?

updates cloud settings/backups, adds restore timestamp

Why is it needed?

updated feature on cloud

@strapi-cla
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@vercel
Copy link

vercel bot commented Nov 19, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
documentation ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 21, 2024 10:24am
@pwizla
Copy link
Collaborator

pwizla commented Nov 20, 2024

Hi @olegStrapier!
Thank you for the contribution.

It seems your PR was "polluted" by a lot of unnecessary formatting changes, probably because you have Prettier or something running on your machine?

To save time, could you please tell me what was the intent of the PR exactly? Only updating the screenshots? Or were other lines of texts changed too?

@pwizla pwizla added the source: Strapi Cloud PRs/issues targeting the Strapi Cloud documentation label Nov 20, 2024
@olegStrapier olegStrapier force-pushed the cloud/backup-restoration-timestamp branch from 69cd77a to f14663b Compare November 21, 2024 10:10
@olegStrapier olegStrapier force-pushed the cloud/backup-restoration-timestamp branch from f14663b to c3d7788 Compare November 21, 2024 10:18
@olegStrapier
Copy link
Contributor Author

hello @pwizla sorry about that I forgot to disable it, it did automatically on push 😦
It's screenshots and I added a new line, new PR committed

@pwizla
Copy link
Collaborator

pwizla commented Nov 21, 2024

Thank you very much for taking the time to clean it up, @olegStrapier 🫶

I also updated our contributing guide, because I noticed yesterday that the line mentioning that linters should be disabled somehow disappeared 😅

@pwizla
Copy link
Collaborator

pwizla commented Nov 21, 2024

I'm merging it now since it seems https://github.com/strapi/cloud/pull/2486 was released yesterday 👀

@pwizla pwizla added this to the 5.2.2 milestone Nov 21, 2024
@pwizla pwizla merged commit 18a0eb1 into main Nov 21, 2024
2 of 3 checks passed
@pwizla pwizla deleted the cloud/backup-restoration-timestamp branch November 21, 2024 14:06
@pwizla pwizla added temp - port to docs-next (Temporary label) The content of this PR should be ported to docs-next.strapi.io and removed temp - port to docs-next (Temporary label) The content of this PR should be ported to docs-next.strapi.io labels Dec 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr: chore source: Strapi Cloud PRs/issues targeting the Strapi Cloud documentation

3 participants