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: Document OsReceive API #987

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Ldoppea
Copy link
Member

@Ldoppea Ldoppea commented Oct 19, 2023

This PR documents the new OsReceive API that is exposed by the Flagship app to the cozy-apps from #906

The cozy-app manifest content is documented in cozy/cozy-stack#4179

Base automatically changed from feat--implement-file-upload-and-communication to master October 26, 2023 12:49
nono added a commit to cozy/cozy-stack that referenced this pull request Nov 2, 2023
This PR documents the new manifest entries that should be used to make a
cozy-app elligible for receiving files from the Flagship app as
implemented in cozy/cozy-flagship-app#906

The OsReceive API is documented in cozy/cozy-flagship-app#987
@Crash--
Copy link
Contributor

Crash-- commented Dec 5, 2023

is it still needed? Can we close it?

@Crash-- Crash-- closed this Dec 5, 2023
@Crash-- Crash-- reopened this Dec 5, 2023
@acezard
Copy link
Contributor

acezard commented Jan 5, 2024

is it still needed? Can we close it?

I think we can remove it, I'm not sure to remember this PR about documentation.
But docs have been merged a while ago https://github.com/cozy/cozy-flagship-app/tree/master/src/app/domain/osReceive/docs

@Ldoppea
Copy link
Member Author

Ldoppea commented Jan 13, 2024

I have to rebase this and check if it is still true after your previous refactorings, but this documents how the API can be used by the cozy-apps. This seems to have not been documented yet (@acezard your link seems to point to Architecture documentations but not usage documentation)

@Ldoppea
Copy link
Member Author

Ldoppea commented Jan 13, 2024

I rebased the PR, but I did not check if the doc is up-to-date yet

@Crash--
Copy link
Contributor

Crash-- commented Feb 9, 2024

ping @Ldoppea

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