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

#2833 Add custom text to Publish Popup #2962

Merged
merged 7 commits into from
Mar 3, 2016

Conversation

sekmiller
Copy link
Contributor

Adds a customizable text to the popup shown to a user when a dataset is published

@sekmiller sekmiller added this to the 4.3 milestone Feb 22, 2016
@scolapasta scolapasta assigned sekmiller and unassigned kcondon Feb 24, 2016
@sekmiller
Copy link
Contributor Author

Related Issues:
#2833

RFI Checklist

  • Functionality completed as described in Issue Ticket
  • Dependencies, risks, assumptions in FRD addressed(N/A - refer to ticket)
  • Unit tests completed (Since mostly interface related integration tests performed)
  • Deployment requirements identified (e.g., SQL scripts, indexing)
  • Documentation completed
  • All code checkins completed
  • Code review completed or waived
  • Testing requirements completed
  • Usability testing completed or waived
  • Support testing completed or waived - waived
  • Merged with develop branch and resolved conflicts

@scolapasta scolapasta assigned kcondon and unassigned sekmiller Feb 24, 2016
@kcondon
Copy link
Contributor

kcondon commented Feb 25, 2016

Passing back to Mike to resolve the remaining popup issue in #2833

@kcondon kcondon assigned mheppler and unassigned kcondon Feb 25, 2016
@mheppler
Copy link
Contributor

Fixed scroll bar issue with Publish Dataset popups on dataset pg.

Sending pull-request and issue back to @kcondon for testing.

@mheppler mheppler assigned kcondon and unassigned mheppler Feb 29, 2016
@kcondon kcondon assigned eaquigley and unassigned kcondon Feb 29, 2016
@kcondon
Copy link
Contributor

kcondon commented Feb 29, 2016

Passing to Liz for User testing.

@pdurbin
Copy link
Member

pdurbin commented Mar 2, 2016

@eaquigley as I mentioned I'd like to use https://beta.dataverse.org soonish for working on #2937. Please let me know when it is available.

@scolapasta scolapasta assigned scolapasta and unassigned eaquigley Mar 3, 2016
scolapasta added a commit that referenced this pull request Mar 3, 2016
@scolapasta scolapasta merged commit e27d3d6 into develop Mar 3, 2016
@mheppler mheppler deleted the 2833-configurable-message-on-publish branch September 27, 2016 18:37
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.

6 participants