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.

Sign up for GitHub

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

Jump to bottom

RMDX - First pass #1497

Draft

jharvey10 wants to merge 127 commits into main from 1492-create-rmdx-processing-microservice

Contributor

jharvey10 commented Nov 7, 2022 •

edited

Loading

(Move these to "Closes" once they are done)
Relates to #1492
Closes #1493
Closes #1494
Relates to #1495
Relates to #1496

First pass at secure RMDX functionality instead of mdx-processor + mdx-sanitizer

TODO List

Remove MDX Processor package
Decide what to do with MDX Sanitizer package
Unit tests for rmdx-processing microservice
Unit tests for rmdx package
Unit tests for api/rmdx
Test docker build of rmdx-processing

Changelog

New

{{ New thing }}

Changed

{{ Changed thing }}

Removed

{{ Removed thing }}

Testing / reviewing

{{ Add descriptions, steps or a checklist for how reviewers can verify this PR works or not }}


          feat(rmdx): create rmdx package skeleton

93531c3

jharvey10 linked an issue

that may be closed by this pull request

Create rmdx-processing microservice #1492

Open

jharvey10 added the status: WIP 👩‍💻 label

jharvey10 and others added 27 commits

November 9, 2022 11:55


          feat(rmdx): first pass (rough draft) rmdx utils

cda77d8


          Merge branch 'main' of github.com:carbon-design-system/carbon-platfor…

13cca8b

…m into 1492-create-rmdx-processing-microservice


          feat(rmdx): support link markdown type

a77b9ba


          feat(rmdx): handle mdxJsxFlowElement in parser

0fe7209


          feat(mdx-components): cleanup some components for component mapper

9a3c9a5

Co-authored-by: Andrea N. Cardona <[email protected]>


          feat(web-app): add some component mdx-components mappers

0f4ab20

Co-authored-by: Andrea N. Cardona <[email protected]>


          Merge branch '1492-create-rmdx-processing-microservice' of github.com…

11bfea9

…:carbon-design-system/carbon-platform into 1492-create-rmdx-processing-microservice


          refactor(rmdx): break node handlers into separate files

b5163a4


          Merge branch '1492-create-rmdx-processing-microservice' of github.com…

1d6f2c8

…:carbon-design-system/carbon-platform into 1492-create-rmdx-processing-microservice


          feat(rmdx): add allowedComponents param to process function

e0f1892


          fix(rmdx): re-add heading as known node type

dfc5e33


          fix(mdx-components): minor props fixes and sb error cleanup

1024d4c

Co-authored-by: Andrea Cardona <[email protected]>


          fix(web-app): divider className fix, use individual mapper per comp

4d2fcce

Co-authored-by: Andrea Cardona <[email protected]>


          feat(mdx-components): cleanup some components for component mapper

991ce0b

Co-authored-by: Andrea N. Cardona <[email protected]>


          feat(web-app): add more component mdx-components mappers

17b8a18

Co-authored-by: Andrea N. Cardona <[email protected]>


          feat(mdx-components): cleanup some components for component mapper

0bf4620

Co-authored-by: Andrea N. Cardona <[email protected]>


          feat(web-app): add more component mdx-components mappers

ef1384a

Co-authored-by: Andrea N. Cardona <[email protected]>


          feat(api): inject runtime into all platform microservices

10cfb5c


          feat(rmdx-processing): microservice skeleton code

f34b472

- Assorted docs updates


          feat(api): remove liveness handler from status-controller

ae6a28d


          Merge branch '1492-create-rmdx-processing-microservice' of github.com…

139dd73

…:carbon-design-system/carbon-platform into 1492-create-rmdx-processing-microservice


          deps: add rmdx-processing to top-level workspaces

3d9d74f

- Improve some "adding a workspace" docs


          docs: add "how to be a good logger" in logging service docs

4fd2c52


          feat(rmdx): add image and blockquote handler, minor rmdx node refactor

e59fc27

Co-authored-by: Joe Harvey <[email protected]>
Co-authored-by: Andrea Cardona <[email protected]>


          fix(mdx-components): remove TODO comment

55e0fee

Co-authored-by: Joe Harvey <[email protected]>
Co-authored-by: Andrea Cardona <[email protected]>


          fix(web-app): change test content

dac8cb7

Co-authored-by: Joe Harvey <[email protected]>
Co-authored-by: Andrea Cardona <[email protected]>


          Merge branch '1492-create-rmdx-processing-microservice' of github.com…

dee70f7

…:carbon-design-system/carbon-platform into 1492-create-rmdx-processing-microservice

jharvey10 and others added 28 commits

December 14, 2022 16:40


          feat(mdx-components): remove re-exported button

783c92c


          fix(mdx-components): anchorlinks column fix and minor api fixes

3f66288


          fix(web-app): migrate color overview to a .js page

7c42b86


          Merge branch '1492-create-rmdx-processing-microservice' of github.com…

27cf7b8

…:carbon-design-system/carbon-platform into 1492-create-rmdx-processing-microservice


          fix(web-app): migrate design get started page from mdx to js

86b9a87


          fix(mdx-components): use full-path import for carbon icons

f190701

- This allows the mdx-components module to work when imported as an es module


          feat(rmdx-processing): add basic rmdx query message impl

3cb6b80


          Merge branch '1492-create-rmdx-processing-microservice' of github.com…

e99e4b1

…:carbon-design-system/carbon-platform into 1492-create-rmdx-processing-microservice


          feat(mdx-components): add custom track component

d189f48


          fix(web-app): add track rmdx test

0c5169f


          feat(api): add rmdx messaging API wrapper

030f51a


          Merge branch '1492-create-rmdx-processing-microservice' of github.com…

93085ed

…:carbon-design-system/carbon-platform into 1492-create-rmdx-processing-microservice


          Merge branch 'main' of github.com:carbon-design-system/carbon-platfor…

00d37f3

…m into 1492-create-rmdx-processing-microservice


          deps: rebuild lock file after merging from main


          deps: install class-transformer and class-validator

30e2aa4

Fixes #1024

- Also updates data-graph docs
- Also fixes unit test issues from merge from main
- Remove class-* deps from esbuild external deps


          fix(mdx-components): add static sb dir, minor track api change

aff9394


          fix(web-app): convert .mdx guidelines/accessibility/color to a js page

03bf86e


          feat(rmdx): add frontmatter to RmdxResponse

4dba065

Co-authored-by: Andrea Cardona <[email protected]>
Co-authored-by: Joe Harvey <[email protected]>


          feat(mdx-components): add fullText prop to h1-h3

cdb18fe

Co-authored-by: Andrea Cardona <[email protected]>
Co-authored-by: Joe Harvey <[email protected]>


          feat(rmdx): add frontmatter, unwrap fn and fullText prop to headings

24cd938

Co-authored-by: Andrea Cardona <[email protected]>
Co-authored-by: Joe Harvey <[email protected]>


          feat(web-app): render remote pages using rmdx processor

2d37ca9

Co-authored-by: Andrea Cardona <[email protected]>
Co-authored-by: Joe Harvey <[email protected]>


          ci: add rmdx docker image workflow

b805c84

This workflow should be removed before this branch is merged in.


          ci: switch rmdx ci to auto-run on branch pushes

e812716


          fix(mdx-components): remove console.log and unnecessar catch statement

11a5f76

Co-authored-by: Andrea Cardona <[email protected]>
Co-authored-by: Joe Harvey <[email protected]>


          feat(web-app): add heading renderers and use custom heading components

404ee0d

Co-authored-by: Andrea Cardona <[email protected]>
Co-authored-by: Joe Harvey <[email protected]>


          fix(web-app): ignore mdx pages for docker build

e804a0e

Co-authored-by: Andrea Cardona <[email protected]>
Co-authored-by: Joe Harvey <[email protected]>


          Merge branch '1492-create-rmdx-processing-microservice' of github.com…

86a77e4

…:carbon-design-system/carbon-platform into 1492-create-rmdx-processing-microservice


          Merge branch 'main' of github.com:carbon-design-system/carbon-platfor…

c50dd32

…m into 1492-create-rmdx-processing-microservice

Contributor

carbon-bot commented Jan 26, 2023

❗ The commits in this PR will result in the following version changes:

🌟 MINOR release: @carbon-platform/api
🌟 MINOR release: @carbon-platform/data-graph
🌟 MINOR release: @carbon-platform/logging
🌟 MINOR release: @carbon-platform/mdx-components
🌟 MINOR release: @carbon-platform/rmdx
🌟 MINOR release: @carbon-platform/rmdx-processing
🌟 MINOR release: @carbon-platform/web-app
🐛 PATCH release: @carbon-platform/icons
🐛 PATCH release: @carbon-platform/mdx-processor
🐛 PATCH release: @carbon-platform/mdx-sanitizer
🐛 PATCH release: @carbon-platform/resources
🐛 PATCH release: @carbon-platform/root
🐛 PATCH release: @carbon-platform/schemas
🐛 PATCH release: @carbon-platform/scripts
🐛 PATCH release: eslint-plugin-carbon-platform
🐛 PATCH release: micromanage-cli

jharvey10 mentioned this pull request

Merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

alisonjoseph Awaiting requested review from alisonjoseph alisonjoseph will be requested when the pull request is marked ready for review alisonjoseph is a code owner

francinelucca Awaiting requested review from francinelucca francinelucca will be requested when the pull request is marked ready for review francinelucca is a code owner

At least 2 approving reviews are required to merge this pull request.

Labels

status: WIP 👩‍💻