Skip to content

chore: upgrade admin dashboard ui to react 18 #1643

chore: upgrade admin dashboard ui to react 18

chore: upgrade admin dashboard ui to react 18 #1643

Triggered via pull request November 24, 2024 03:27
@lwhiteleylwhiteley
edited #740
Status Success
Total duration 1m 5s
Artifacts

pr-lint.yml

on: pull_request_target
Validate & Label PR
6s
Validate & Label PR
Fit to window
Zoom out
Zoom in

Annotations

2 errors
Validate & Label PR
Resource not accessible by integration { name: 'HttpError', id: '11992764634', status: 403, response: { url: 'https://api.github.com/repos/finos/git-proxy/releases', status: 403, headers: { 'access-control-allow-origin': '*', 'access-control-expose-headers': 'ETag, Link, Location, Retry-After, X-GitHub-OTP, X-RateLimit-Limit, X-RateLimit-Remaining, X-RateLimit-Used, X-RateLimit-Resource, X-RateLimit-Reset, X-OAuth-Scopes, X-Accepted-OAuth-Scopes, X-Poll-Interval, X-GitHub-Media-Type, X-GitHub-SSO, X-GitHub-Request-Id, Deprecation, Sunset', connection: 'close', 'content-encoding': 'gzip', 'content-security-policy': "default-src 'none'", 'content-type': 'application/json; charset=utf-8', date: 'Sun, 24 Nov 2024 03:28:29 GMT', 'referrer-policy': 'origin-when-cross-origin, strict-origin-when-cross-origin', server: 'github.com', 'strict-transport-security': 'max-age=31536000; includeSubdomains; preload', 'transfer-encoding': 'chunked', vary: 'Accept-Encoding, Accept, X-Requested-With', 'x-accepted-github-permissions': 'contents=write; contents=write,workflows=write', 'x-content-type-options': 'nosniff', 'x-frame-options': 'deny', 'x-github-api-version-selected': '2022-11-28', 'x-github-media-type': 'github.v3; format=json', 'x-github-request-id': '243A:161458:3D7E77:76DE9B:67429D5D', 'x-ratelimit-limit': '5000', 'x-ratelimit-remaining': '4992', 'x-ratelimit-reset': '1732422507', 'x-ratelimit-resource': 'core', 'x-ratelimit-used': '8', 'x-xss-protection': '0' }, data: { message: 'Resource not accessible by integration', documentation_url: 'https://docs.github.com/rest/releases/releases#create-a-release', status: '403' } }, request: { method: 'POST', url: 'https://api.github.com/repos/finos/git-proxy/releases', headers: { accept: 'application/vnd.github.v3+json', 'user-agent': 'probot/12.2.5 octokit-core.js/3.5.1 Node.js/20.13.1 (linux; x64)', authorization: 'token [REDACTED]', 'content-type': 'application/json; charset=utf-8' }, body: `{"target_commitish":"refs/heads/main","name":"Version 1.7.1","tag_name":"v1.7.1","body":"### What's Changed\\n\\n* No changes\\n\\n---\\n\\n*Full Changelog**: https://github.com/finos/git-proxy/compare/v1.7.0...v1.7.1\\n","draft":true,"prerelease":false,"make_latest":"true"}`, request: {} }, event: { id: '11992764634', name: 'pull_request_target', payload: { action: 'edited', changes: { body: { from: 'Changes:\r\n' + '- upgrade to react 18\r\n' + '- upgrade to mui v6\r\n' + ' - increase specificity of app styles to override mui component styles \r\n' + '- replace react-html-parser with html-react-parser\r\n' + '\r\n' + 'Notes:\r\n' + '\r\n' + 'After an audit of the dependencies of this project, it was determined that react and a few libraries were outdated.\r\n' + '\r\n' + 'one of the dependencies preventing an upgrade is `react-html-parser` . it has a peer dependency of react 16.\r\n' + 'A successor of the package is `html-react-parser`\r\n' + '\r\n' + 'However, there is no storybook setup in the project or a way to test this easily.\r\n' + '\r\n' + 'How can i get diff data passed to this component to test in isolation or \r\n' + 'Can someone give steps to arrive at this view to test it in the UI to verify it works?\r\n' + '\r\n' + 'I also see that there are cypress tests but i dont believe this view is covered yet\r\n' + '\r\n' + 'old package: https://www.npmjs.com/package/react-html-parser\r\n' + 'proposed package: https://www.npmjs.com/package/html-react-parser\r\n' + '\r\n' + '----\r\n' + '\r\n' + 'TODOs:\r\n
Validate & Label PR
HttpError: Resource not accessible by integration at /home/runner/work/_actions/release-drafter/release-drafter/v6/dist/index.js:8462:21 at process.processTicksAndRejections (node:internal/process/task_queues:95:5) at async Job.doExecute (/home/runner/work/_actions/release-drafter/release-drafter/v6/dist/index.js:30793:18) { name: 'AggregateError', event: { id: '11992764634', name: 'pull_request_target', payload: { action: 'edited', changes: { body: { from: 'Changes:\r\n' + '- upgrade to react 18\r\n' + '- upgrade to mui v6\r\n' + ' - increase specificity of app styles to override mui component styles \r\n' + '- replace react-html-parser with html-react-parser\r\n' + '\r\n' + 'Notes:\r\n' + '\r\n' + 'After an audit of the dependencies of this project, it was determined that react and a few libraries were outdated.\r\n' + '\r\n' + 'one of the dependencies preventing an upgrade is `react-html-parser` . it has a peer dependency of react 16.\r\n' + 'A successor of the package is `html-react-parser`\r\n' + '\r\n' + 'However, there is no storybook setup in the project or a way to test this easily.\r\n' + '\r\n' + 'How can i get diff data passed to this component to test in isolation or \r\n' + 'Can someone give steps to arrive at this view to test it in the UI to verify it works?\r\n' + '\r\n' + 'I also see that there are cypress tests but i dont believe this view is covered yet\r\n' + '\r\n' + 'old package: https://www.npmjs.com/package/react-html-parser\r\n' + 'proposed package: https://www.npmjs.com/package/html-react-parser\r\n' + '\r\n' + '----\r\n' + '\r\n' + 'TODOs:\r\n' + '- [x] fix ui styling glitches \r\n' + ' - increase css specificity \r\n' + '- [x] fix dep check\r\n' + '\r\n' + 'Please test thoroughly\r\n' + 'Let me know if i missed something' } }, number: 740, organization: { avatar_url: 'https://avatars.githubusercontent.com/u/35377814?v=4', description: 'FINOS’ mission is to promote open innovation in financial services. See our full list of repos from our nearly 100 projects & 11 programs at finos.github.io', events_url: 'https://api.github.com/orgs/finos/events', hooks_url: 'https://api.github.com/orgs/finos/hooks', id: 35377814, issues_url: 'https://api.github.com/orgs/finos/issues', login: 'finos', members_url: 'https://api.github.com/orgs/finos/members{/member}', node_id: 'MDEyOk9yZ2FuaXphdGlvbjM1Mzc3ODE0', public_members_url: 'https://api.github.com/orgs/finos/public_members{/member}', repos_url: 'https://api.github.com/orgs/finos/repos', url: 'https://api.github.com/orgs/finos' }, pull_request: { _links: { comments: { href: 'https://api.github.com/repos/finos/git-proxy/issues/740/comments' }, commits: { href: 'https://api.github.com/repos/finos/git-proxy/pulls/740/commits' }, html: { href: 'https://github.com/finos/git-proxy/pull/740' }, issue: { href: 'https://api.github.com/repos/finos/git-proxy/issues/740' }, review_comment: { href: 'https://api.github.com/repos/finos/git-proxy/pulls/comments{/number}' }, review_comments: { href: 'https://api.github.com/repos/finos/git-proxy/pulls/740/comments' }, self: { href: 'https://api.github.com/repos/finos/git-proxy/pulls/740' }, statuses: { href: 'https://api.github.com/repos/finos/git-proxy/statuses/742409a30aa81a6ddfd744190fe5d5238edab7ed' } },