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

Create initial project structure (Frontend) #10

Merged
merged 35 commits into from
Nov 9, 2023
Merged

Create initial project structure (Frontend) #10

merged 35 commits into from
Nov 9, 2023

Conversation

archeoss
Copy link
Contributor

Partly resolves the #1 issue

Depends on #9. Merging that request will ease the review of this PR

@archeoss archeoss linked an issue Aug 25, 2023 that may be closed by this pull request
Create initial project structure #1
Closed
@archeoss archeoss mentioned this pull request Sep 3, 2023
Merged
@archeoss archeoss mentioned this pull request Sep 5, 2023
Merged
@ikopylov
Copy link
Member

ikopylov commented Sep 7, 2023

In general looks good, but there are many changes from previous request (#9), so it is hard to look it in details. After #9 will be merged, I will review this PR again

@archeoss archeoss mentioned this pull request Sep 19, 2023
Merged
@archeoss archeoss changed the base branch from main to 1-init-project-backend October 2, 2023 23:30
@archeoss
Copy link
Contributor Author

archeoss commented Oct 2, 2023 •
edited
Loading

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @archeoss and the rest of your teammates on Graphite Graphite

@archeoss archeoss mentioned this pull request Oct 3, 2023
Merged
ikopylov
ikopylov reviewed Oct 4, 2023
View reviewed changes
CHANGELOG.md Outdated Show resolved Hide resolved
ikopylov
ikopylov reviewed Oct 4, 2023
View reviewed changes
README.md Show resolved Hide resolved
ikopylov
ikopylov reviewed Oct 4, 2023
View reviewed changes
build.rs Outdated Show resolved Hide resolved
ikopylov
ikopylov reviewed Oct 4, 2023
View reviewed changes
backend/Cargo.toml Outdated Show resolved Hide resolved
@archeoss archeoss mentioned this pull request Oct 5, 2023
Open
@ikopylov ikopylov changed the base branch from 1-init-project-backend to main October 5, 2023 14:45
@ikopylov
Copy link
Member

ikopylov commented Oct 5, 2023

Please, resolve merge conflicts

ikopylov
ikopylov reviewed Oct 5, 2023
View reviewed changes
backend/src/main.rs Outdated Show resolved Hide resolved
ikopylov
ikopylov reviewed Oct 5, 2023
View reviewed changes
frontend/build.rs Outdated Show resolved Hide resolved
ikopylov
ikopylov reviewed Oct 5, 2023
View reviewed changes
frontend/build.rs Outdated Show resolved Hide resolved
ikopylov
ikopylov reviewed Oct 5, 2023
View reviewed changes
frontend/build.rs Outdated Show resolved Hide resolved
ikopylov
ikopylov reviewed Oct 17, 2023
View reviewed changes
backend/src/main.rs Outdated Show resolved Hide resolved
ikopylov
ikopylov reviewed Oct 19, 2023
View reviewed changes
backend/src/router.rs Outdated Show resolved Hide resolved
ikopylov
ikopylov reviewed Oct 19, 2023
View reviewed changes
backend/src/router.rs Outdated Show resolved Hide resolved
ikopylov
ikopylov reviewed Oct 20, 2023
View reviewed changes
backend/src/main.rs Outdated Show resolved Hide resolved
ikopylov
ikopylov reviewed Oct 20, 2023
View reviewed changes
backend/src/router.rs Outdated Show resolved Hide resolved
ikopylov
ikopylov reviewed Oct 20, 2023
View reviewed changes
backend/src/router.rs Outdated Show resolved Hide resolved
ikopylov
ikopylov reviewed Oct 20, 2023
View reviewed changes
backend/src/router.rs Outdated Show resolved Hide resolved
ikopylov
ikopylov reviewed Oct 20, 2023
View reviewed changes
backend/src/router.rs Outdated Show resolved Hide resolved
ikopylov
ikopylov reviewed Oct 26, 2023
View reviewed changes
backend/Cargo.toml Outdated Show resolved Hide resolved
ikopylov
ikopylov reviewed Oct 26, 2023
View reviewed changes
backend/src/router.rs Outdated Show resolved Hide resolved
ikopylov
ikopylov approved these changes Nov 9, 2023
View reviewed changes
Copy link
Member

@ikopylov ikopylov left a comment

Choose a reason for hiding this comment

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

LGTM

@ikopylov ikopylov merged commit 6ab8e31 into main Nov 9, 2023
This was referenced Nov 15, 2023
Open
Open
Open
Open
This was referenced Jan 29, 2024
Open
Open
Open
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ikopylov ikopylov ikopylov approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Create initial project structure
2 participants
@archeoss @ikopylov