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

[chores] UX: added loading indicator when VPN server is created #227 #229

Merged
merged 1 commit into from Jul 8, 2020

Conversation

nemesifier
Copy link
Member

After 2.5 seconds a text which asks the user to wait is shown.
We need this because generating the Diffie Helman of 2048 bits
length takes considerable more time.

Closes #227

@nemesifier nemesifier requested a review from atb00ker July 7, 2020 19:12
@nemesifier nemesifier self-assigned this Jul 7, 2020
@nemesifier nemesifier added this to In progress in OpenWISP Priorities for next releases via automation Jul 7, 2020
@nemesifier nemesifier added this to In progress in [GSoC20] Merge modules via automation Jul 7, 2020
@nemesifier nemesifier moved this from In progress to Open Pull-Requests in [GSoC20] Merge modules Jul 7, 2020
@coveralls
Copy link

coveralls commented Jul 7, 2020

Coverage Status

Coverage increased (+0.001%) to 98.867% when pulling 17820b7 on 227-vpn-creation-loading-indicator into dff8658 on master.

Copy link
Member

@atb00ker atb00ker left a comment

Choose a reason for hiding this comment

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

Hi, the Post request is not send when the form is filled! 😄

openwisp_controller/config/static/config/js/vpn.js Outdated Show resolved Hide resolved
spinner.fadeOut(250, function () {
var message = gettext(
'Please be patient, we are creating all the necessary ' +
'cyrptographic keys which may take a minute or two ...'
Copy link
Member

Choose a reason for hiding this comment

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

Also, vscode wants to change indentation of this line. :-P

Copy link
Member Author

Choose a reason for hiding this comment

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

jslinter is complaining, maybe we can try switching to jshint

After 2.5 seconds a text which asks the user to wait is shown.
We need this because generating the Diffie Helman of 2048 bits
length takes considerable more time.

Closes #227
@nemesifier nemesifier force-pushed the 227-vpn-creation-loading-indicator branch from e739171 to 17820b7 Compare July 8, 2020 19:32
@nemesifier nemesifier merged commit 17820b7 into master Jul 8, 2020
OpenWISP Priorities for next releases automation moved this from In progress to Done Jul 8, 2020
[GSoC20] Merge modules automation moved this from Open Pull-Requests to Done Jul 8, 2020
@nemesifier nemesifier deleted the 227-vpn-creation-loading-indicator branch July 8, 2020 19:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

[admin] Add loading indicator on VPN server first save
3 participants