Skip to content

Conversation

@ssundahlTTD
Copy link
Contributor

@ssundahlTTD ssundahlTTD commented Dec 23, 2024

  • rename server-only to server-side to be more accurate
  • correctly link to client-side integration example for client-server sample page
  • ensure links to source are correct (they will be when deployed)
  • change titles to more correctly describe the type of sample
  • clean up wording

href="https://unifiedid.com/docs/guides/integration-google-ss#server-side-integration"
>server-side UID2 integration</a
href="https://unifiedid.com/docs/guides/integration-google-ss#sdk-for-javascript-client-side-integration"
>client-side UID2 integration</a
Copy link
Contributor

Choose a reason for hiding this comment

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

Just checking - these changes have client-server and client-side - is this expected?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's a good question. I think so. The link calls out the client-side integration example but the overall example is client-server since the server is generating the token but the client is initializing the sdk with the identity from the server.

Copy link
Contributor

Choose a reason for hiding this comment

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

i dont think this is correct -

short answer:
it should be referring to Client-Server Integration and better to link to this instead as a quick fix:
https://unifiedid.com/docs/guides/integration-javascript-client-server

long answer:
We should add a Client-Server Integration option section in GAM Secure Signals instead of linking to SDK for JavaScript Client-Side Integration

So in GAM Secure Signal page, we should mention 3 integration methods - Server-Side, Client-Server and Client-Side. cc @genwhittTTD on this but probably it is best to talk thru between Gen/Scott/Me after new year.

@ssundahlTTD ssundahlTTD merged commit 194d4dd into main Dec 23, 2024
3 checks passed
@ssundahlTTD ssundahlTTD deleted the sas-UID2-3917-sample-site-text-links branch December 23, 2024 23:22
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.

4 participants