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

Project specific id's #1752

Closed
canstudios-louisem opened this issue Oct 25, 2017 · 2 comments
Closed

Project specific id's #1752

canstudios-louisem opened this issue Oct 25, 2017 · 2 comments
Assignees
Labels
D: insane Not for the fainthearted, require considerable discussion S: in-progress Issues currently being worked on - leave these be! T: enhancement Adding additional functionality

Comments

@canstudios-louisem
Copy link
Contributor

canstudios-louisem commented Oct 25, 2017

When copying a course any components or styling that use specific id's break

Affected Area

Course Copying

Requested Feature

No longer use the _id on ABC's but a _courseId that is not regenerated when the course is copied

Additional Information

We have written a detailed proposal on how we would like to tackle this https://docs.google.com/document/d/17JNGj6szijXtxHcFehNLXfi18D2jO4DHcMHeXuf2c5g/edit?usp=sharing

@canstudios-louisem canstudios-louisem added the D: insane Not for the fainthearted, require considerable discussion label Oct 25, 2017
@canstudios-louisem canstudios-louisem added this to the 0.5.0 milestone Oct 25, 2017
@canstudios-louisem canstudios-louisem added the T: enhancement Adding additional functionality label Oct 30, 2017
@canstudios-louisem canstudios-louisem removed this from the 0.5.0 milestone Oct 30, 2017
@canstudios-louisem canstudios-louisem self-assigned this Apr 6, 2018
@canstudios-louisem canstudios-louisem added the S: in-progress Issues currently being worked on - leave these be! label Apr 6, 2018
@taylortom taylortom added this to the 0.7.0 Loose ends milestone Nov 19, 2018
@Janooba
Copy link

Janooba commented Nov 7, 2019

I'd like to add my support for this feature, as it would be extremely important for our team. We currently do not use the authoring tool because of this alone. The ID's are not human-readable and once exported, manually editing the course (as its often required for us) becomes nearly impossible.

If I may make a suggestion, I'd like the generated _id's to follow a similar readable standard to the following.

The goal of this naming convention is to meet the following criteria. It must be:

  • Concise (As small and easy to type as possible)
  • Readable (Should be distinct and simple to read)
  • Self-documenting (Should trace back exactly where this element is located within the structure)
Content Object: [#]
Article: [A-Z]
_ (Underscore)
Block: [#]
Component: [a|b]

Examples:

  • 01B_16a
  • 02A_04b

Our team has been using this structure to manually develop Adapt courses for a while now with great success.

Cheers!

@taylortom
Copy link
Member

wontfix for legacy, will be introduced in v1.0. Some discussion here adapt-security/adapt-authoring-ui#193

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
D: insane Not for the fainthearted, require considerable discussion S: in-progress Issues currently being worked on - leave these be! T: enhancement Adding additional functionality
Projects
None yet
Development

No branches or pull requests

3 participants