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.

Sign up for GitHub

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

Jump to bottom

Add docblock to update function from Drupal. #37

Closed
jenlampton opened this issue Aug 22, 2016 · 0 comments
Closed

Add docblock to update function from Drupal. #37

jenlampton opened this issue Aug 22, 2016 · 0 comments
Milestone
1.x-1.1.3

Comments

@jenlampton
Copy link
Member

When updating from Drupal to backdrop all you see in the UI for the webform update is

webform module
    1000 -

which can be a little scary. By adding a docblock to the update function users will know what is about to happen to their webforms!' How about something like this?

/**
 * Update webforms from Drupal to Backdrop.
 */
@herbdool herbdool mentioned this issue Jan 19, 2017
Closed
herbdool added a commit that referenced this issue Jan 23, 2017
@herbdool
Issue #37: adds docblock for update
e9768a3 
And correct spacing of code.
@herbdool herbdool modified the milestone: 1.x-1.1.3 Jan 24, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
1.x-1.1.3
Development

No branches or pull requests
2 participants
@jenlampton @herbdool