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

Implement a simple forking option #97

Closed
ehmatthes opened this issue Aug 3, 2015 · 3 comments
Closed

Implement a simple forking option #97

ehmatthes opened this issue Aug 3, 2015 · 3 comments

Comments

@ehmatthes
Copy link
Member

Implement a simple forking option when creating a new school, so new users can use one of the site's main strengths - not having to reinvent the wheel. Copy all elements, but no editor relationships.

  • What are the critical tests to write?
  • Where do the links to do this exist?
  • What's the workflow for doing this?
@ehmatthes
Copy link
Member Author

  • Critical tests: (see Write more tests for views.fork #103)
    • Rejects a non-empty organization.
    • Check that only public organizations can be forked.
    • Fork an organization, and make sure all elements in the new organization match the old organization.
    • Check there's nothing in new org that's not in old org.
    • Make sure original org is unchanged, in all aspects.
    • Add a new sa to old organization, make sure it's in old organization and not in new organization.
    • Add new sa to new organization, make sure it's in new organization and not in old organization.
    • Make sure user forking is the only editor of the new organization.
  • Further tests:
    • Make sure only public elements are in new org. (Actually, easy enough to do in checking that systems match.)

@ehmatthes
Copy link
Member Author

  • UI/ links
    • User creates a new organization.
    • On the organization page, after stating no elements, link to fork.
    • On forking page, can only fork public orgs now. (Later, could fork private orgs that you have edit access on.)
    • utils.fork() only copies public elements.
    • [ ] Will need to report errors in forking process to user. later (fork() reports errors to user #100)

@ehmatthes
Copy link
Member Author

Rest of the tests moved to #103.

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

No branches or pull requests

1 participant