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

feat: update render markdown for tables #4390

Merged
merged 2 commits into from Aug 28, 2023

Conversation

luka-nextcloud
Copy link
Contributor

πŸ“ Summary

πŸ–ΌοΈ Screenshots

🏚️ Before 🏑 After
image image

🚧 TODO

  • ...

🏁 Checklist

  • Code is properly formatted (npm run lint / npm run stylelint / composer run cs:check)
  • Sign-off message is added to all commits
  • Tests (unit, integration and/or end-to-end) passing and the changes are covered with tests
  • Documentation (README or documentation) has been updated or is not required

@cypress
Copy link

cypress bot commented Jun 26, 2023

1 failed and 3 flaky tests on run #11873 β†—οΈŽ

1 132 18 0 Flakiness 3

Details:

feat: update render markdown for tables
Project: Text Commit: e109f0ea74
Status: Failed Duration: 04:10 πŸ’‘
Started: Aug 27, 2023 8:03 PM Ended: Aug 27, 2023 8:07 PM
FailedΒ  cypress/e2e/attachments.spec.js β€’ 1 failed test

View Output Video

Test Artifacts
Test all attachment insertion methods > test if attachment folder is deleted after having deleted a markdown file Output Screenshots
FlakinessΒ  share.spec.js β€’ 2 flaky tests

View Output Video

Test Artifacts
Open test.md in viewer > Shares the file as a public read only link Output Screenshots
Open test.md in viewer > Share a file with download disabled shows an error Output Screenshots
FlakinessΒ  api/SyncServiceProvider.spec.js β€’ 1 flaky test

View Output Video

Test Artifacts
Sync service provider > recovers from a dropped message Output Screenshots

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

@juliushaertl
Copy link
Member

Cypress failure looks unrelated, but can you check the jest tests again @luka-nextcloud ?

@luka-nextcloud luka-nextcloud force-pushed the feature/render-markdown-for-tables branch from dc55e9c to e3bb72d Compare June 26, 2023 17:13
@juliushaertl juliushaertl requested a review from mejo- June 27, 2023 07:08
@juliushaertl juliushaertl added this to the Nextcloud 28 milestone Jun 27, 2023
@juliushaertl juliushaertl added the enhancement New feature or request label Jun 27, 2023
@juliushaertl juliushaertl removed the request for review from mejo- July 19, 2023 10:11
@max-nextcloud
Copy link
Collaborator

funny that the node check is passing without a compile.

@max-nextcloud
Copy link
Collaborator

/backport e3bb72d to stable27

@max-nextcloud
Copy link
Collaborator

/compile

@max-nextcloud max-nextcloud force-pushed the feature/render-markdown-for-tables branch from 7a4fff2 to c587f55 Compare August 27, 2023 19:42
@max-nextcloud
Copy link
Collaborator

/compile

luka-nextcloud and others added 2 commits August 28, 2023 05:02
Signed-off-by: Luka Trovic <luka@nextcloud.com>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@max-nextcloud max-nextcloud force-pushed the feature/render-markdown-for-tables branch from e109f0e to 1c168c0 Compare August 28, 2023 03:02
@max-nextcloud max-nextcloud merged commit ddad1d7 into main Aug 28, 2023
32 checks passed
@max-nextcloud max-nextcloud deleted the feature/render-markdown-for-tables branch August 28, 2023 03:11
@backportbot-nextcloud
Copy link

The backport to stable27 failed. Please do this backport manually.

# Switch to the target branch and update it
git checkout stable27
git pull origin stable27

# Create the new backport branch
git checkout -b fix/foo-stable27

# Cherry pick the change from the commit sha1 of the change against the default branch
# This might cause conflicts. Resolve them.
git cherry-pick abc123

# Push the cherry pick commit to the remote repository and open a pull request
git push origin fix/foo-stable27

More info at https://docs.nextcloud.com/server/latest/developer_manual/getting_started/development_process.html#manual-backport

@juliushaertl
Copy link
Member

@luka-nextcloud Can you take care of the manual backport please?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review enhancement New feature or request
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Format markdown code for tables
4 participants