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

[Rpc][update doc] apply rpc ppx syntax #23

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

Conversation

GuerfelMohamed
Copy link

No description provided.

@balat
Copy link
Member

balat commented Oct 13, 2020

There are still many occurrences of server_function in tutorials. Do I need to do your documentation myself?
Also Eliom's documentation is not up to date.

tutos/dev/manual/how-to-call-a-server-side-function-from-client-side.wiki:let%client f_rpc = ~%(Eliom_client.server_function [%derive.json: int] f)
tutos/dev/manual/how-to-make-the-client-side-program-get-an-html-element-from-the-server-and-insert-it-in-the-page.wiki:  ~%(server_function [%derive.json: unit] get_mydiv)
tutos/dev/manual/how-to-send-a-file-to-server-without-stopping-the-client-process.wiki:there is a way to send it to the server. You can't use the {{{server_function}}} syntax for this, but you can use
tutos/dev/manual/tutoreact.wiki:use {{{server_function}}}:
tutos/dev/manual/tutoreact.wiki:  Eliom_client.server_function
tutos/dev/manual/tutoreact.wiki:  Eliom_client.server_function [%derive.json: int]
tutos/dev/manual/tutoreact.wiki:  Eliom_client.server_function
tutos/dev/manual/tutoreact.wiki:  Eliom_client.server_function
tutos/dev/manual/tutoreact.wiki:  Eliom_client.server_function [%derive.json: int]
tutos/dev/manual/tutoreact.wiki:  Eliom_client.server_function
tutos/dev/manual/tutoreact.wiki:  Eliom_client.server_function [%derive.json: int]
tutos/dev/manual/tutoreact.wiki:  Eliom_client.server_function [%derive.json: int]

@balat
Copy link
Member

balat commented Oct 13, 2020

Do not merge this until:

  • All occurrences of server_function in tuto are translated
  • Eliom doc is updated
  • OS template and doc are updated

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants