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

Component library #516

Merged
merged 28 commits into from
Nov 20, 2023
Merged

Component library #516

merged 28 commits into from
Nov 20, 2023

Conversation

aliceoq
Copy link
Contributor

@aliceoq aliceoq commented Oct 26, 2023

What is the purpose of this pull request?

To use the components from the component library.

How should this be manually tested?

Open the preview and check if the following components are working in the same way as the current Developers Portal:

  • Sidebar
  • Search input
  • Search page
  • Feedback section + feedback modal
  • Documentation page (markdown renderer)
  • Table of Contents

More information on here.

Screenshots or example usage

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Requires change to documentation, which has been updated accordingly.

@netlify
Copy link

netlify bot commented Oct 26, 2023

Deploy Preview for elated-hoover-5c29bf ready!

Name Link
🔨 Latest commit b14d2c2
🔍 Latest deploy log https://app.netlify.com/sites/elated-hoover-5c29bf/deploys/655b3b953604b90008888953
😎 Deploy Preview https://deploy-preview-516--elated-hoover-5c29bf.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@aliceoq aliceoq added the release-minor Minor version bump label Oct 26, 2023
@github-actions
Copy link

github-actions bot commented Oct 26, 2023

A total of 6 tests failed!

API guides documentation page

2 failing tests:

  • Check if a random guide page, chosen using the sidebar, loads (/docs/guides)
  • try to click on any document contributor (/docs/guides)

Status of documentation pages

4 failing tests:

  • Checks page docs/guides/webservice-deprecated
  • Checks page docs/guides/webservice-deprecated
  • Checks page docs/guides/webservice-deprecated
  • Checks page docs/guides/webservice-deprecated

For more information, open the cypress action summary page.

@aliceoq aliceoq requested a review from RobsonOlv October 26, 2023 16:25
@aliceoq aliceoq force-pushed the feat/sidebar-module branch from 2d03cab to cbb7905 Compare October 31, 2023 10:58
@aliceoq aliceoq marked this pull request as ready for review October 31, 2023 13:46
@aliceoq aliceoq marked this pull request as draft October 31, 2023 13:46
@aliceoq aliceoq force-pushed the feat/sidebar-module branch 2 times, most recently from 3585538 to 7bb2799 Compare November 8, 2023 21:13
@aliceoq aliceoq requested a review from brunoamui November 8, 2023 21:25
@aliceoq aliceoq force-pushed the feat/sidebar-module branch from 4c952a0 to 6b1b99d Compare November 9, 2023 00:41
@aliceoq aliceoq marked this pull request as ready for review November 9, 2023 11:17
@carolinamenezes
Copy link
Contributor

Callouts não estão renderizando corretamente:

Comportamento do preview Comportamento desejado

Atentar para o fato que temos comportamentos diferentes para os seguintes casos:

❗ This is a Danger callout.

⚠ This is a Warning callout.

✅ This is a Success callout.

ℹ️ This is an Info callout.

Sendo o callout do estilo ℹ️, o default

@carolinamenezes
Copy link
Contributor

Estilos do mermaid foram motificados:

Estilo do preview Estilo desejado

@carolinamenezes
Copy link
Contributor

Estilos de código estão diferentes:

Estilo no preview Estilo desejado

@carolinamenezes
Copy link
Contributor

Placeholder da barra de pesquisa ficou descentralizado

Comportamento do preview Comportamento desejado

@carolinamenezes
Copy link
Contributor

O componente <Flex> era exportado e usado em algumas docs, mas acho que dá para resolver adaptando o componente de Card, sem precisar exportar o Flex:

Comportamento do preview Comportamento desejado

@carolinamenezes
Copy link
Contributor

carolinamenezes commented Nov 10, 2023

Placeholder da pesquisa da sidebar não indica que a pesquisa serve apenas para o menu selecionado

Comportamento do preview Comportamento desejado

ps.: tem um erro na versão (in in) em prod 😮

Copy link
Contributor

@carolinamenezes carolinamenezes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@RobsonOlv RobsonOlv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚀

Copy link
Collaborator

@brunoamui brunoamui left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@aliceoq aliceoq force-pushed the feat/sidebar-module branch from ad01b6b to b14d2c2 Compare November 20, 2023 10:57
@aliceoq aliceoq merged commit a3b563d into main Nov 20, 2023
6 of 7 checks passed
@aliceoq aliceoq deleted the feat/sidebar-module branch November 20, 2023 11:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-minor Minor version bump
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants