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.

Sign up for GitHub

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

Jump to bottom

design(frontend): AGE-513 Implement Overview and App management view #1986

Merged
merged 127 commits into from
Aug 21, 2024
Merged

design(frontend): AGE-513 Implement Overview and App management view #1986

merged 127 commits into from
Aug 21, 2024

Conversation

bekossy
Copy link
Member

@bekossy bekossy commented Aug 13, 2024

Description

This pull request focuses on enhancing the user interface and user experience of the app management and overview pages.

@bekossy
design(frontend): added antd theme config tokens
274ba56
@bekossy
feat(frontend): installed phosphor-icons package
57ab4da
@bekossy
feat(frontend): setup font family and antd tokens
657af2d
@bekossy
design(frontend): implement sidebar design
47a8d02
@bekossy
fix(frontend): modified logo size and improved sidebar in oss
d6fd55d
@bekossy
minor improvements
f872795
@bekossy
design(frontend): added menu item header
c87473b
@bekossy
fix(frontend): changed submenu icons
034e460
@bekossy
design(frontend): added overview route
c8b0c6c
@bekossy
fix(frontend): revert logo image width and height
00cda03
@bekossy
fix(frontend): modified endpoint from apps page to overview
63bfd00
@bekossy
fix(frontend): export createParams Func for reusability
594f38a
@bekossy
design(frontend): minor layout
5a4e839
@bekossy
design(frontend): style deployment section in overview page
b6501d0
@bekossy
fix(frontend): implemented api logic and minor improvements
67e7adf
@bekossy
design(frontend): added variants overview section
12b97ac
@bekossy
fix(frontend): modified invite modal query param value
de4bc40
@bekossy
design(frontend): added all overview sections
588ba1f
@bekossy
design(frontend): added query param key value to open evaluation moda…
e2b069a 
…ls on navigate
@bekossy
design(frontend): drafted variants overview
193798e
@bekossy
fix(frontend): dynamically imported Observability overview
d59c7d1
@bekossy
fix(frontend): added transformer function to human eval response
f9024f9
@bekossy
fix(frontend): added settings option in evaluation table
a880b50
@bekossy
fix(frontend): improved logic and added card loaders
25259c3
@bekossy
fix(frontend): improved ab testing eval overview table
03124b2
@bekossy
fix(frontend): improved single model eval overview table
6005089
@bekossy
fix(frontend): removed query human eval modal
e533e3c
@bekossy
design(frontend): modified deployment overview drawer width and added…
82e12bd 
… version and cleanup in Layout
@bekossy
minor changes
8404d63
@bekossy
fix(frontend): added day format for human eval table
2bac31d
jp-agenta
jp-agenta approved these changes Aug 21, 2024
View reviewed changes
Copy link
Contributor

@jp-agenta jp-agenta left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Code review: lgtm :)
QA: @aybruhm @bekossy -- Could you please add a full QA checklist, including (some) relevant regression testing and (all) relevant feature(s) testing ?

@mmabrouk
Copy link
Member

@jp-agenta This has already been done in the Linear issue. The QA has also been done yesterday. (relevant discussion)

@jp-agenta
Copy link
Contributor

jp-agenta commented Aug 21, 2024 •
edited by aybruhm
Loading

@jp-agenta This has already been done in the Linear issue. The QA has also been done yesterday. (relevant discussion)

Then it's a go.

@jp-agenta jp-agenta merged commit 6ebd073 into main Aug 21, 2024
13 of 14 checks passed
@jp-agenta jp-agenta deleted the prototype-design/-project-revamp branch August 21, 2024 11:09
@bekossy bekossy restored the prototype-design/-project-revamp branch August 21, 2024 14:03
@bekossy bekossy deleted the prototype-design/-project-revamp branch August 21, 2024 21:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aybruhm aybruhm aybruhm left review comments

@mmabrouk mmabrouk mmabrouk approved these changes

@jp-agenta jp-agenta jp-agenta approved these changes

Assignees
No one assigned
Labels
Frontend lgtm This PR has been approved by a maintainer size:XXL This PR changes 1000+ lines, ignoring generated files. UI UX
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@bekossy @mmabrouk @jp-agenta @aybruhm @ashrafchowdury