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

Feature/tas 13 layout #43

Merged

Conversation

markgcera
Copy link
Contributor

Pull Request

Description

Hi Xinming, this PR is coming from a branch which branched off from main, which is why there are so many changed files. The only file I added is the 'layout.tsx' file which contains commented out code for the layout container, with comments on how to use the container.

Related Issues

[Reference any related issues that this pull request addresses or resolves.]

Checklist

Please review and check the following before submitting your pull request:

  • I have read the CONTRIBUTING.md document.
  • My code follows the code style of this project.
  • I have written tests for my changes (if applicable).
  • All new and existing tests passed.
  • I have updated the documentation accordingly (if applicable).
  • I have added comments to my code, particularly in areas that may be confusing.

Screenshots (if applicable)

[Include any relevant screenshots or GIFs to visually demonstrate the changes.]

Desktop

Mobile

Additional Information

[Include any additional information that might be helpful for reviewers or maintainers.]

Reviewer Notes

[Optional: Include specific instructions or notes for reviewers, if necessary.]

@markgcera markgcera requested a review from gnimnix March 19, 2024 13:38
@gnimnix
Copy link
Contributor

gnimnix commented Mar 25, 2024

@markgcera , next time can do a merge from DEV-4 then resolve any merge conflicts.

…omponents-and-layouts' into feature/TAS-13-Layout

# Conflicts:
#	src/routes/App.tsx
@gnimnix gnimnix merged commit 30f32ce into epic/DEV-4-shared-components-and-layouts May 22, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants