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: move Views to Eppie.App.Shared #381

Conversation

al-kau
Copy link
Collaborator

@al-kau al-kau commented Oct 23, 2024

Issue

Related to #347

Description

Move files from Views to Eppie.App.Shared.
Сomment out some unsupported items.
Fix the xaml and build errors.
Correct styles: file encoding, whitespace formatting, imports ordering.

Other Information

UI task #382

Pull Request Checklist

Please check if your pull request fulfills the following requirements:

Note

Types of pull requests
  • feat - adding new features
  • fix - bug fixes
  • test - adding or correcting tests
  • perf - changes that improve performance
  • refactor - simple rewriting or restructuring of code without adding new features or fixing bugs
  • style - changes in code styles and no changes in logic
  • build - changes related to the build of the project and dependencies
  • ci - changes related to continuous integration
  • docs - changes in documentation or just comments in source code
  • chore - something that doesn't fit the other possible types
Pull request labels
  • type/breaking-change - pull requests with changes that are not backward compatible
  • type/build - pull requests that change the project's build or dependencies
  • type/chore - pull requests without making changes to the code, project build, formatting, documentation, etc
  • type/ci - pull requests whose changes are related to continuous integration
  • type/documentation - pull requests that only change documentation
  • type/feature - pull requests that add new features
  • type/fix - pull requests that fix a bug
  • type/localization - pull requests that change translation
  • type/performance - pull requests that improve performance
  • type/refactor - pull requests that refactor a section of code
  • type/style - pull requests that change code styles
  • type/test - pull requests that add or correct tests
  • ignore-for-release - for pull requests that do not need to be appeared in release notes

A pull request appears in the release notes if it has one of the labels: type/breaking-change, type/feature, type/fix, type/localization, type/performance

Affected platforms labels
  • platform/all - pull requests that are related to the all platforms
  • platform/android - pull requests that are related to the Android platform
  • platform/desktop - pull requests that are related to the desktop
  • platform/ios - pull requests that are related to the iOS platform
  • platform/macos - pull requests that are related to the macOS platform
  • platform/other - pull requests that are related to an unknown platform
  • platform/uwp - pull requests that are related to the Universal Windows Platform
  • platform/wasm - pull requests that are related to the WebAssembly platform
  • platform/winui - pull requests that are related to the WinUI platform

- Fix whitespace formatting.
- Fix file encoding.
- Fix imports ordering.

References: Eppie-io#347
@al-kau al-kau added platform/all Categorizes an issue or PR as relevant to the all platforms type/chore Pull requests without making changes to code, project build, formatting, documentation, etc labels Oct 23, 2024
@al-kau al-kau requested a review from ValeraFinebits October 23, 2024 15:58
@ValeraFinebits ValeraFinebits merged commit 10dae15 into Eppie-io:feature/296-upgrading-to-uno-platform-5 Oct 23, 2024
5 checks passed
@al-kau al-kau deleted the feature/347-shared-code-views branch October 24, 2024 13:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
platform/all Categorizes an issue or PR as relevant to the all platforms type/chore Pull requests without making changes to code, project build, formatting, documentation, etc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants