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

Updating Talawa-Docs with DartDoc markdown files fails #1768

Closed
palisadoes opened this issue Apr 12, 2023 · 7 comments · Fixed by #1772
Closed

Updating Talawa-Docs with DartDoc markdown files fails #1768

palisadoes opened this issue Apr 12, 2023 · 7 comments · Fixed by #1772
Assignees
Labels
bug Something isn't working documentation Update documentation

Comments

@palisadoes
Copy link
Contributor

Describe the bug
We recognized that Talawa-Docs was not getting updated as expected in this issue:

However, documentation is not getting updated as expected. The push workflow created is not working and needs to be rectified:

To Reproduce

All our pushes are failing. This needs to be fixed.

Expected behavior

See Above

Actual behavior

See Above

Screenshots

N/A

Additional details

N/A

@palisadoes palisadoes added the bug Something isn't working label Apr 12, 2023
@github-actions github-actions bot added documentation Update documentation unapproved Unapproved, needs to be triaged labels Apr 12, 2023
@literalEval
Copy link
Member

@palisadoes on it :)

@palisadoes palisadoes removed the unapproved Unapproved, needs to be triaged label Apr 12, 2023
@literalEval
Copy link
Member

literalEval commented Apr 12, 2023

@palisadoes so I read this in detail. Have you allowed "force push" or any other bypassing method to the admin account ?

@palisadoes
Copy link
Contributor Author

We got it working with this GitHub action.

The settings for all reps are the same and should be OK.

@literalEval
Copy link
Member

@palisadoes that's what the current push.yml is doing in this repo. Does that mean it will be working fine now ?

@palisadoes
Copy link
Contributor Author

The current push.yml file doesn't use the TALAWA_DOCS_SYNC environment variable nor the TALAWA_ENVIRONMENT environment variable. Therefore it may not be working correctly. Please work with @EshaanAgg to verify whether what you currently have is correct.

@literalEval
Copy link
Member

literalEval commented Apr 12, 2023

Oh I forgot to open the PR. I meant that my current local implementation whose screenshot I showed as a working example, is working fine and is using these environment secrets. The updated file can be seen here (I had changed the repo's name to check its working on my fork).

With these changes, it should work right ?

@literalEval
Copy link
Member

@palisadoes I am opening a PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Update documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants