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

Created integration page, and pre-fill registration article #4

Merged
merged 7 commits into from Jan 6, 2014

Conversation

spirosikmd
Copy link
Contributor

The introductory text in the Integration page will be written later today.

:slug: articles/prefill-registration
:tags: integration

*In some cases it is required that the registration form in the front office is pre-filled
Copy link
Contributor

Choose a reason for hiding this comment

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

I think it is nicer to start a bit more commercially relevant, as it is written also for merchants/third parties. So something like this: "Paylogic offers a feature to allow the pre-filling of the personal data of a consumer during the sales process via a personalized URL. This can be used whenever a merchant already has the data of the consumers and wants to offer these consumers a more personalized and smooth ticketing process, or when a pre-registration step is used. The following sections...".

:slug: articles/prefill-registration
:tags: integration

*Paylogic offers a feature to allow the pre-filling of the personal data of a
Copy link
Contributor

Choose a reason for hiding this comment

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

'the pre-filling' => 'prefilling'
'the personal data' => 'personal data'

the length of the data, as the URL should contain up to a maximum number of
characters. It is not only that the JSON data itself might be long. Encoding
also increases the length of the data by approximately 33%. Encoding is used to
protect the privacy of the data.
Copy link
Contributor

Choose a reason for hiding this comment

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

Encoding does in no way protect the privacy of the data. Implying that Paylogic thinks it does is, in my opinion, potentially damaging to the company.

xolox added a commit that referenced this pull request Jan 6, 2014
@xolox xolox merged commit 6eefb4b into paylogic:master Jan 6, 2014
@xolox
Copy link
Member

xolox commented Jan 6, 2014

Thanks for the article Spyros! Your pull request has been merged and the article has been published: developer.paylogic.com/articles/prefill-registration.html.

I mentioned previously (after a local preview of the merge result) that something strange was going on with the code snippet in the first section ("JSON data format"). Now that the article is live I still see the same problem: The first code snippet simply disappears off the page (although it's still visible in the DOM tree through the inspector). This only happens in Chromium for me; in Firefox it looks fine.

Also, I made some minor changes, please refer to the merge diff for the details (I didn't change any content, just RST format details).

@bubenkoff
Copy link
Contributor

could we update homepage with last articles?
to have it like blog

On 6 January 2014 17:31, Peter Odding notifications@github.com wrote:

Thanks for the article Spyros! Your pull request has been merged and the
article has been published:
developer.paylogic.com/articles/prefill-registration.html.

I mentioned previously (after a local preview of the merge result) that
something strange was going on with the code snippet in the first section
("JSON data format"). Now that the article is live I still see the same
problem: The first code snippet simply disappears off the page (although
it's still visible in the DOM tree through the inspector). This only
happens in Chromium for me; in Firefox it looks fine.

Also, I made some minor changes, please refer to the merge diff for the
details (I didn't change any content, just RST format details).


Reply to this email directly or view it on GitHubhttps://github.com//pull/4#issuecomment-31662274
.

Anatoly Bubenkov

@xolox
Copy link
Member

xolox commented Jan 6, 2014

@bubenkoff You mean that /index.html contains the latest article instead of linking to it? This can probably be done by customizing Pelican, although I'm not sure how to approach it.

@bubenkoff
Copy link
Contributor

yes, to look more like blog, because it should be a blog, right?

On 6 January 2014 17:36, Peter Odding notifications@github.com wrote:

@bubenkoff https://github.com/bubenkoff You mean that /index.htmlcontains the latest article instead of linking to it? This can probably be
done by customizing Pelican, although I'm not sure how to approach it.


Reply to this email directly or view it on GitHubhttps://github.com//pull/4#issuecomment-31662780
.

Anatoly Bubenkov

@maikelwever
Copy link
Contributor

Something like this @xolox and @bubenkoff: #5?

@maikelwever
Copy link
Contributor

And about those code highlights: on Chrome on Linux it works fine (v31), but with older firefox (only one I have installed right now: v23) the code dissapears.

@xolox
Copy link
Member

xolox commented Jan 7, 2014

I mentioned previously (after a local preview of the merge result) that something strange was going on with the code snippet in the first section ("JSON data format"). Now that the article is live I still see the same problem: The first code snippet simply disappears off the page (although it's still visible in the DOM tree through the inspector). This only happens in Chromium for me; in Firefox it looks fine.

The mystery on my end was solved when I merged and previewed #5: the Chrome extension JSON Prettifier was to blame. I noticed this when I previewed the result of #5 and saw that the JSON block jumped to the bottom of the page, outside the normal page flow, and included some funky nonstandard bits of layout. When I disabled the extension everything went back to normal :-)

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.

None yet

7 participants