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

Chore: Update to 6.0.0 #1820

Merged
merged 40 commits into from
Jun 15, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
40 commits
Select commit Hold shift + click to select a range
32cfa71
chore(release): 4.3.0 (#852)
ElinorW Feb 16, 2021
a42b742
chore(release): 4.4.0 (#860)
ElinorW Feb 22, 2021
8190568
chore(release): 4.4.1 (#878)
thewahome Mar 3, 2021
bc65ddf
chore(release): 4.4.2 (#912)
thewahome Mar 30, 2021
5368580
chore(release): 4.5.0 (#929)
thewahome Apr 13, 2021
d690cd8
Fix linting errors preventing the pipeline from runnings (#930)
thewahome Apr 13, 2021
69758cb
chore(release): 4.6.0 (#938)
thewahome May 5, 2021
9107145
Set GET as default HTTP method if method specified in query params is…
millicentachieng May 13, 2021
40d8316
Task: Update version number (#951)
millicentachieng May 13, 2021
e0ef4a4
chore(release): 4.9.0 (#989)
thewahome Jun 10, 2021
c477432
chore(release): 4.10.0 (#1014)
thewahome Jul 6, 2021
19733ac
chore(release): 4.11.0 (#1031)
thewahome Jul 15, 2021
c919328
chore(release): 4.12.0 (#1046)
thewahome Aug 3, 2021
d16607c
Release: September 2021 (#1091)
thewahome Sep 1, 2021
edfb717
Release: November 2021 (#1238)
ElinorW Nov 18, 2021
932a672
Release: December 2021 (#1308)
Onokaev Dec 9, 2021
194610a
Merge branch 'dev'
Onokaev Jan 5, 2022
f167a60
Revert "Merge branch 'dev'"
Onokaev Jan 5, 2022
1ff4ea0
Release 01-2022 (#1344)
millicentachieng Jan 13, 2022
0b3facd
Release: February 2022 (#1417)
ElinorW Feb 3, 2022
4be0d11
Chore (release): 4.17.0 (#1549)
github-actions[bot] Mar 10, 2022
8e825b1
Chore (release): 4.18.0 (#1553)
github-actions[bot] Mar 11, 2022
d47dd98
Chore (patch): 4.18.1 (#1558)
thewahome Mar 15, 2022
fef5627
Chore (release): 5.0.0 (#1582)
github-actions[bot] Mar 30, 2022
bb595f7
Chore (release): 5.1.0 (#1662)
github-actions[bot] Apr 13, 2022
bc54fb4
Chore (release): 5.1.1 (#1678)
github-actions[bot] Apr 22, 2022
d523741
Chore (release): 5.2.0 (#1729)
github-actions[bot] May 16, 2022
f8be28d
Bump version to 6.0.0
invalid-email-address Jun 9, 2022
8cb8faa
Merge remote-tracking branch 'origin/master' into release/6.0.0
ElinorW Jun 9, 2022
18f4628
Merge branch 'master' into release/6.0.0
ElinorW Jun 10, 2022
7dba302
remove unused files
ElinorW Jun 10, 2022
7c050df
remove menu beak
ElinorW Jun 13, 2022
5b2cdff
reduce width and height
ElinorW Jun 13, 2022
4d29d30
remove title
ElinorW Jun 14, 2022
91880a6
Update from https://github.com/microsoftgraph/microsoft-graph-devx-co…
thewahome Jun 14, 2022
2a7ce41
Update from https://github.com/microsoftgraph/microsoft-graph-devx-co…
thewahome Jun 14, 2022
16c26c0
Update from https://github.com/microsoftgraph/microsoft-graph-devx-co…
thewahome Jun 14, 2022
d12b032
Update from https://github.com/microsoftgraph/microsoft-graph-devx-co…
thewahome Jun 14, 2022
8f77fa7
Update from https://github.com/microsoftgraph/microsoft-graph-devx-co…
thewahome Jun 14, 2022
505c4d9
Update from https://github.com/microsoftgraph/microsoft-graph-devx-co…
thewahome Jun 14, 2022
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
52 changes: 25 additions & 27 deletions .github/workflows/linter.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ jobs:
- name: Set up Node
uses: actions/setup-node@v3
with:
node-version: '14.x'
node-version: "14.x"

- name: Cache npm
uses: actions/cache@v3
Expand Down Expand Up @@ -52,26 +52,26 @@ jobs:
runs-on: ubuntu-latest

steps:
- name: Checkout repository
uses: actions/checkout@v3
with:
fetch-depth: 0

- name: Set up Node
uses: actions/setup-node@v3
with:
node-version: '14.x'

- name: Cache node_modules
uses: actions/cache@v3
with:
path: |
node_modules
packages/*/node_modules
key: modules-${{ hashFiles('package-lock.json') }}

- name: Lint
run: npm run lint
- name: Checkout repository
uses: actions/checkout@v3
with:
fetch-depth: 0

- name: Set up Node
uses: actions/setup-node@v3
with:
node-version: "14.x"

- name: Cache node_modules
uses: actions/cache@v3
with:
path: |
node_modules
packages/*/node_modules
key: modules-${{ hashFiles('package-lock.json') }}

- name: Lint
run: npm run lint

build:
name: Build packages
Expand All @@ -85,7 +85,7 @@ jobs:
- name: Set up Node
uses: actions/setup-node@v3
with:
node-version: '14.x'
node-version: "14.x"

- name: Cache node_modules
uses: actions/cache@v3
Expand Down Expand Up @@ -113,7 +113,7 @@ jobs:
- name: Set up Node
uses: actions/setup-node@v3
with:
node-version: '14.x'
node-version: "14.x"

- name: Cache node_modules
uses: actions/cache@v3
Expand Down Expand Up @@ -156,11 +156,9 @@ jobs:
- name: Run sonar cloud analysis
uses: SonarSource/sonarcloud-github-action@master
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} # Needed to get PR information, if any
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} # Needed to get PR information, if any
SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}



publish-preview-site:
name: Static preview site creation
needs: install
Expand Down Expand Up @@ -199,4 +197,4 @@ jobs:
uses: Azure/static-web-apps-deploy@v0.0.1-preview
with:
azure_static_web_apps_api_token: ${{ secrets.AZURE_STATIC_WEB_APPS_API_TOKEN_JOLLY_SAND_0AC78C710 }}
action: "close"
action: "close"
2 changes: 1 addition & 1 deletion package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "graph-explorer-v2",
"version": "5.2.0",
"version": "6.0.0",
"private": true,
"dependencies": {
"@augloop/types-core": "file:packages/types-core-2.16.189.tgz",
Expand Down
2 changes: 1 addition & 1 deletion src/app/utils/query-parameter-sanitization.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -291,4 +291,4 @@ describe('Sanitize Query Parameters should', () => {
});
});

});
});
6 changes: 2 additions & 4 deletions src/app/utils/query-parameter-sanitization.ts
Original file line number Diff line number Diff line change
Expand Up @@ -433,8 +433,7 @@ function sanitizeFilterQueryOptionValue(queryParameterValue: string): string {
if (!isPropertyName(propertyName)) {
propertyName = '<property>';
}
sanitizedQueryString += `${queryFunctionPrefix}(${propertyName}${
commaIndex > 0 ? ',<value>' : ''
sanitizedQueryString += `${queryFunctionPrefix}(${propertyName}${commaIndex > 0 ? ',<value>' : ''
})`;
} else {
sanitizedQueryString += `${queryFunctionPrefix}(<unknown>)`;
Expand All @@ -460,8 +459,7 @@ function sanitizeFilterQueryOptionValue(queryParameterValue: string): string {
COMPARISON_OPERATORS.includes(expectedOperator) ||
ARITHMETIC_OPERATORS.includes(expectedOperator)
) {
sanitizedQueryString += `${segment} ${
filterSegments[index + 1]
sanitizedQueryString += `${segment} ${filterSegments[index + 1]
} <value>`;
index += 2;
continue;
Expand Down
38 changes: 38 additions & 0 deletions src/app/views/app-sections/FeedbackButton.spec.tsx
Original file line number Diff line number Diff line change
@@ -0,0 +1,38 @@
import React from 'react';
import { cleanup, render } from '@testing-library/react';

import { FeedbackButton } from './FeedbackButton';

afterEach(cleanup);
const renderFeedbackButton = () => {
return render(
<FeedbackButton />
)
}

jest.mock('react-redux', () => ({
useSelector: jest.fn(() => {
return {
profile: {
profileType: 'MSA'
}
}
}),
useDispatch: jest.fn()
}));

// eslint-disable-next-line no-console
console.warn = jest.fn();

// eslint-disable-next-line react/display-name
jest.mock('../query-runner/request/feedback/FeedbackForm.tsx', () => () => {
return <div>Feedback</div>
})

describe('Tests Feedback button', () => {
it('Renders feedback button without crashing', () => {
const { getByText } = renderFeedbackButton();
expect(getByText(/Help Improve Graph Explorer/)).toBeDefined();

});
})
67 changes: 67 additions & 0 deletions src/app/views/app-sections/FeedbackButton.tsx
Original file line number Diff line number Diff line change
@@ -0,0 +1,67 @@
import { DirectionalHint, IconButton, IIconProps, TooltipHost } from '@fluentui/react';
import React, { useState } from 'react';
import { translateMessage } from '../../utils/translate-messages';
import { useSelector } from 'react-redux';
import FeedbackForm from '../query-runner/request/feedback/FeedbackForm';
import { IRootState } from '../../../types/root';
import { ACCOUNT_TYPE } from '../../services/graph-constants';

export const FeedbackButton = () => {
const [enableSurvey, setEnableSurvey] = useState(false);
const { profile } = useSelector( (state: IRootState) => state );

const feedbackIcon : IIconProps = {
iconName : 'Feedback'
}
const feedbackTitle = translateMessage('Feedback');
const content = <div style={{padding:'3px'}}>{translateMessage('Feedback')}</div>

const feedbackIconStyles = {
root:{
height: '50px',
width: '50px'
}
}
const calloutProps = {
gapSpace: 0
};
const hostStyles = { root: {
display: 'inline-block'
}
};

const toggleSurvey = () => {
setEnableSurvey(prevState => !prevState);
}

const disableSurvey = () => {
setEnableSurvey(false);
}

return (
<div>
{profile?.profileType !== ACCOUNT_TYPE.AAD &&
<div>
<TooltipHost
content={content}
calloutProps={calloutProps}
styles={hostStyles}
directionalHint={DirectionalHint.leftCenter}
>
<IconButton onClick={toggleSurvey}
iconProps={feedbackIcon}
ariaDescription={feedbackTitle}
ariaLabel={feedbackTitle}
styles={feedbackIconStyles}
role={'button'}
disabled={enableSurvey}
/>
</TooltipHost>

<FeedbackForm onDismissSurvey={toggleSurvey}
activated={enableSurvey} onDisableSurvey={disableSurvey} />
</div>
}
</div>
)
}
2 changes: 1 addition & 1 deletion src/app/views/app-sections/HeaderMessaging.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ export function headerMessaging(classes: any, query: string): React.ReactNode {
const loginType = getLoginType();

return (
<div style={{ marginBottom: 8 }}>
<div style={{ marginBottom: 8, paddingLeft: 10 }}>
{loginType === LoginType.Popup && <>
<MessageBar messageBarType={MessageBarType.info} isMultiline={true}>
<p>
Expand Down
2 changes: 1 addition & 1 deletion src/app/views/authentication/profile/Profile.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -197,7 +197,7 @@ const Profile = (props: any) => {
role='dialog'
gapSpace={0}
target={`#${buttonId}`}
isBeakVisible={true}
isBeakVisible={false}
beakWidth={10}
onDismiss={toggleIsCalloutVisible}
setInitialFocus
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,6 @@ export const ShareQuery = () => {
styles={shareButtonStyles}
role={'button'}
ariaLabel={translateMessage('Share Query')}
title={translateMessage('Share Query')}
/>
</TooltipHost>
<Dialog
Expand Down
4 changes: 2 additions & 2 deletions src/app/views/sidebar/Sidebar.styles.ts
Original file line number Diff line number Diff line change
Expand Up @@ -118,8 +118,8 @@ export const sidebarStyles = (theme: ITheme) => {
},
sidebarButtons: {
root:{
height: 50,
width: 50,
height: 40,
width: 40,
':hover': {
background: `${theme.palette.neutralLight} !important`
}
Expand Down
1 change: 0 additions & 1 deletion src/app/views/sidebar/resource-explorer/ResourceLink.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,6 @@ const ResourceLink = (props: IResourceLinkProps) => {
aria-describedby={tooltipId}
styles={iconButtonStyles}
menuIconProps={{ iconName: 'MoreVertical' }}
title={translateMessage('More actions')}
menuProps={{
shouldFocusOnMount: true,
alignTargetEdge: true,
Expand Down
1 change: 0 additions & 1 deletion src/app/views/sidebar/sample-queries/SampleQueries.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -151,7 +151,6 @@ const unstyledSampleQueries = (sampleProps?: ISampleQueriesProps): JSX.Element =
>
<Icon
iconName='Lock'
title={signInText}
style={{
fontSize: 15,
height: 10,
Expand Down
18 changes: 12 additions & 6 deletions src/messages/GE_de-DE.json
Original file line number Diff line number Diff line change
Expand Up @@ -12,9 +12,9 @@
"secure scores (beta)": "Sicherheitsbewertungen (Beta)",
"secure score control profiles (beta)": "Profile für Steuerung der Sicherheitsbewertung (Beta)",
"update alert": "Warnung aktualisieren",
"To try the explorer, please ": "Um den Explorer auszuprobieren, bitte ",
"sign in": "Beim Graph Explorer anmelden",
" with your work or school account from Microsoft.": " mit Ihrem Geschäfts-, Schul- oder Unikonto von Microsoft.",
"To try the explorer, please ": "Um den Explorer auszuprobieren, gehen Sie bitte wie folgt vor ",
"sign in": "Anmelden",
" with your work or school account from Microsoft.": " mit Ihrem Arbeits- oder Schulkonto von Microsoft.",
"Submit": "Senden",
"Using demo tenant": "Sie verwenden derzeit ein Beispielkonto.",
"To access your own data:": "Melden Sie sich an, um auf Ihre eigenen Daten zuzugreifen.",
Expand Down Expand Up @@ -301,8 +301,8 @@
"Microsoft Graph API Reference docs": "Microsoft Graph-API-Referenzdokumenten.",
"Fetching permissions": "Berechtigungen werden abgerufen",
"Authentication failed": "Authentifizierung fehlgeschlagen",
"view all permissions": "Berechtigungen auswählen",
"Fetching code snippet": "Code-Schnipsel abrufen",
"view all permissions": "Zustimmung zu Berechtigungen",
"Fetching code snippet": "Codeausschnitt wird abgerufen",
"Snippet not available": "Snippet nicht verfügbar",
"Select different permissions": "Um verschiedene Microsoft Graph API-Endpunkte auszuprobieren, wählen Sie die Berechtigungen aus und klicken Sie dann auf Zustimmen.",
"Search permissions": "Suchberechtigungen",
Expand Down Expand Up @@ -441,5 +441,11 @@
"Switch to beta": "Zur Betaversion wechseln",
"Resources": "Ressourcen",
"More items": "Weitere Elemente",
"Update": "Aktualisierung"
"Update": "Aktualisierung",
"Edit request header": "Anforderungsheader bearbeiten",
"Remove request header": "Anforderungsheader entfernen",
"Settings": "Einstellungen",
"sign in other account": "Mit einem anderen Konto anmelden",
"Help": "Hilfe",
"Documentation": "Dokumentation"
}
12 changes: 9 additions & 3 deletions src/messages/GE_es-ES.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
"secure score control profiles (beta)": "perfiles de control de puntuación segura (beta)",
"update alert": "actualizar alerta",
"To try the explorer, please ": "Para probar el explorador, ",
"sign in": "Inicie sesión en el Probador de Graph",
"sign in": "inicie sesión",
" with your work or school account from Microsoft.": " con su cuenta profesional o educativa de Microsoft.",
"Submit": "Enviar",
"Using demo tenant": "Actualmente utiliza una cuenta de muestra.",
Expand Down Expand Up @@ -301,7 +301,7 @@
"Microsoft Graph API Reference docs": "Documentos de referencia de la API de Microsoft Graph.",
"Fetching permissions": "Obteniendo permisos",
"Authentication failed": "Error de autenticación",
"view all permissions": "Seleccionar permisos",
"view all permissions": "Consentimiento para los permisos",
"Fetching code snippet": "Recuperando fragmento de código",
"Snippet not available": "Fragmento de código no disponible",
"Select different permissions": "Para probar otros puntos de conexión de la API de Microsoft Graph, elija los permisos y, luego, haga clic en Consentir.",
Expand Down Expand Up @@ -441,5 +441,11 @@
"Switch to beta": "Cambiar a beta",
"Resources": "Recursos",
"More items": "Más elementos",
"Update": "Actualizar"
"Update": "Actualizar",
"Edit request header": "Editar el encabezado de la solicitud",
"Remove request header": "Eliminar el encabezado de la solicitud",
"Settings": "Configuración",
"sign in other account": "Inicie sesión con una cuenta diferente",
"Help": "Ayuda",
"Documentation": "Documentación"
}
Loading