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

[docs-infra] Split feedback channels per product #42413

Merged
merged 3 commits into from
May 28, 2024

Conversation

alexfauquette
Copy link
Member

@alexfauquette alexfauquette commented May 27, 2024

Fix mui/mui-public#168

@mui/docs-infra Tested on docs-feedback-material-ui and docs-feedback-base-ui. Will need to be merge to test for X channels

@alexfauquette alexfauquette added the scope: docs-infra Specific to the docs-infra product label May 27, 2024
@alexfauquette alexfauquette self-assigned this May 27, 2024
@mui-bot
Copy link

mui-bot commented May 27, 2024

Netlify deploy preview

https://deploy-preview-42413--material-ui.netlify.app/

Bundle size report

No bundle size changes (Toolpad)
No bundle size changes

Generated by 🚫 dangerJS against e36e403


const BASE_UI_FEEBACKS_CHANNEL_ID = 'C075LJG1LMP';
const MATERIAL_UI_FEEBACKS_CHANNEL_ID = 'C0757QYLK7V';
// const PIGMENT_CSS_FEEBACKS_CHANNEL_ID = 'C074TBW0JKZ';
Copy link
Member Author

Choose a reason for hiding this comment

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

@brijeshb42 I assume we don't have pages for now

Copy link
Contributor

@brijeshb42 brijeshb42 May 27, 2024

Choose a reason for hiding this comment

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

Yeah. No pages yet.

const X_GRID_FEEBACKS_CHANNEL_ID = 'C0757R0KW67';
const X_CHARTS_FEEBACKS_CHANNEL_ID = 'C0757UBND98';
const X_EXPLORE_FEEBACKS_CHANNEL_ID = 'C074TBYQK2T';
// const DESIGN_KITS_FEEBACKS_CHANNEL_ID = 'C075ADGN0UU';
Copy link
Member Author

Choose a reason for hiding this comment

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

@oliviertassinari Do you have a specific page in mind, or is it just an idea for latter for the design kits? Because for now, they don't have pages with feedback input. Or at least I did not find them

Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member

@oliviertassinari oliviertassinari left a comment

Choose a reason for hiding this comment

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

Oh, ok, I didn't expected we would act on the issue already, but if the team already feels like the time for this is right 👍

if (url.includes('/x/react-date-pickers') || url.includes('/x/react-tree-view')) {
return X_EXPLORE_FEEBACKS_CHANNEL_ID;
}
if (url.includes('/x/react-data-grid')) {
Copy link
Member

@oliviertassinari oliviertassinari May 27, 2024

Choose a reason for hiding this comment

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

This doesn't work, e.g. https://mui.com/x/api/data-grid/data-grid/. Is there a way we could rely on productId rather than URLs? For example, once we move the docs to subdomains, this will break.

@alexfauquette
Copy link
Member Author

Oh, ok, I didn't expected we would act on the issue already, but if the team already feels like the time for this is right 👍

The team is mostly me bothered by all the data grid feedback I don't care about 😅

Comment on lines 12 to 17
| 'x-date-pickers'
| 'x-charts';
| 'x-charts'
| 'x-tree-view'
| 'toolpad-studio'
| 'toolpad-core';

Copy link
Member Author

Choose a reason for hiding this comment

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

I don't know why it's in master but not in next

@alexfauquette alexfauquette requested a review from a team May 28, 2024 09:19
Copy link
Contributor

@danilo-leal danilo-leal left a comment

Choose a reason for hiding this comment

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

Looks great! I'm assuming System-related comments are falling into the Material UI channel, right? (I guess that's already the case, isn't it? 🤔)

@alexfauquette alexfauquette merged commit 79bad36 into mui:next May 28, 2024
22 checks passed
@oliviertassinari oliviertassinari added the enhancement This is not a bug, nor a new feature label May 28, 2024
joserodolfofreitas pushed a commit to joserodolfofreitas/material-ui that referenced this pull request Jul 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement This is not a bug, nor a new feature scope: docs-infra Specific to the docs-infra product
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[docs-infra] Split Slack channel per team
5 participants