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

♻️(frontend) replace Paragraph grommet #2435

Merged
merged 1 commit into from
Oct 4, 2023

Conversation

AntoLC
Copy link
Contributor

@AntoLC AntoLC commented Sep 27, 2023

Purpose

Replace the grommet Paragraph component by our custom component Text component base on the Cunningham design system.

Proposal

  • Replace Paragraph component

@AntoLC AntoLC self-assigned this Sep 27, 2023
@AntoLC AntoLC changed the base branch from master to feature/anthony/replace-text-grommet September 27, 2023 12:59
@AntoLC AntoLC force-pushed the feature/anthony/replace-paragraph-grommet branch from a7e1736 to fbc746b Compare September 27, 2023 14:10
@AntoLC AntoLC marked this pull request as ready for review September 27, 2023 14:31
@AntoLC AntoLC force-pushed the feature/anthony/replace-text-grommet branch 7 times, most recently from f14cfc0 to ade02f7 Compare October 4, 2023 09:06
Base automatically changed from feature/anthony/replace-text-grommet to master October 4, 2023 09:22
@AntoLC AntoLC force-pushed the feature/anthony/replace-paragraph-grommet branch from 89ef131 to b1df180 Compare October 4, 2023 09:46
We replace the Grommet Paragraph component with our own Text component
based on the Cunnigham design system.
@AntoLC AntoLC force-pushed the feature/anthony/replace-paragraph-grommet branch from b1df180 to ad457b1 Compare October 4, 2023 09:47
@AntoLC AntoLC merged commit e524c9f into master Oct 4, 2023
33 of 34 checks passed
@AntoLC AntoLC deleted the feature/anthony/replace-paragraph-grommet branch October 4, 2023 10:07
lunika added a commit that referenced this pull request Oct 19, 2023
Added

- Generic video LTI url
- Display complete error message in frontend
- Generic classroom LTI url
- Generic LTI cartridge (xml configuration)
- Display LTI warning message for generic urls regarding course copy
- configure homepage banner using SiteConfig model
- setting BBB_INVITE_TOKEN_BANNED_LIST
- Add transcode_pipeline property to video model

Changed

- Replace ngrok with localtunnel to serve marsha on a public domain with
  stable urls and allow to run multiple tunnels at the same time
- Replace all the loaders by Cunningham Loader (#2436)
- Replace grommet Text (#2427)
- Replace grommet Paragraph (#2435)
- Remove cross on login image (#2451)
- Replace grommet Checkbox and Pagination (#2437)

Fixed

- Markdown save sent previously saved content
- Force to end BBB meeting when creation fail
lunika added a commit that referenced this pull request Oct 19, 2023
Added

- Generic video LTI url
- Display complete error message in frontend
- Generic classroom LTI url
- Generic LTI cartridge (xml configuration)
- Display LTI warning message for generic urls regarding course copy
- configure homepage banner using SiteConfig model
- setting BBB_INVITE_TOKEN_BANNED_LIST
- Add transcode_pipeline property to video model

Changed

- Replace ngrok with localtunnel to serve marsha on a public domain with
  stable urls and allow to run multiple tunnels at the same time
- Replace all the loaders by Cunningham Loader (#2436)
- Replace grommet Text (#2427)
- Replace grommet Paragraph (#2435)
- Remove cross on login image (#2451)
- Replace grommet Checkbox and Pagination (#2437)

Fixed

- Markdown save sent previously saved content
- Force to end BBB meeting when creation fail
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants