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

Bug: script create-cohort-project can't add OAS #1506

Closed
reloadercf opened this issue Jul 31, 2023 · 1 comment · Fixed by #1507
Closed

Bug: script create-cohort-project can't add OAS #1506

reloadercf opened this issue Jul 31, 2023 · 1 comment · Fixed by #1507
Assignees
Labels
bug Error que necesita arreglo

Comments

@reloadercf
Copy link
Contributor

reloadercf commented Jul 31, 2023

The next img by @AlexisJoselyn print the error
image (13)
then I try run the script and the result is likewise in my computer.
Screen Shot 2023-07-31 at 17 08 30

@reloadercf reloadercf changed the title Bug: script create-cohort-project cant Bug: script create-cohort-project can't ADD OAS Jul 31, 2023
@reloadercf reloadercf changed the title Bug: script create-cohort-project can't ADD OAS Bug: script create-cohort-project can't add OAS Jul 31, 2023
@unjust
Copy link
Contributor

unjust commented Aug 1, 2023

Me parece porque los objetivos ahora estan en 10 (un numero de 2 digitos) el regex no funciona.

Cambie linea 170 a
line => /^## \d+ \. Objetivos de aprendiza(je|gem)/i.test(line), y ahora si.

unjust added a commit to unjust/curriculum that referenced this issue Aug 1, 2023
@unjust unjust linked a pull request Aug 1, 2023 that will close this issue
@unjust unjust self-assigned this Aug 1, 2023
@unjust unjust added the bug Error que necesita arreglo label Aug 1, 2023
mfdebian pushed a commit that referenced this issue Aug 1, 2023
* fix: change regex for oa heading fixes #1506

* quita linea vacio
@unjust unjust closed this as completed Aug 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Error que necesita arreglo
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants