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

Enterview: show full category name with delimiter #143

Merged
merged 6 commits into from
Oct 14, 2024
Merged

Conversation

guanlisheng
Copy link
Contributor

this would be part of #5

@guanlisheng guanlisheng requested a review from georgeef October 13, 2024 15:48
@georgeef
Copy link
Contributor

MMEX Desktop uses the separator defined in Infotable (or ":" as default).

@georgeef
Copy link
Contributor

PRs give errors last few days, although they compile and run.

@guanlisheng
Copy link
Contributor Author

Regarding the delimiter config, it will retrieve the delimiter from the infotable.

i think the build errors are from

/Users/runner/work/mmex-ios/mmex-ios/MMEX/Repository/RepositoryProtocol.swift:15:37: error: cannot suppress conformances here
enum RepositoryPluckResult<Wrapped: ~Copyable>: ~Copyable {
                                    ^
                                    
/Users/runner/work/mmex-ios/mmex-ios/MMEX/Repository/RepositoryProtocol.swift:15:38: error: cannot find type 'Copyable' in scope
enum RepositoryPluckResult<Wrapped: ~Copyable>: ~Copyable {
                                     ^~~~~~~~
/Users/runner/work/mmex-ios/mmex-ios/MMEX/Repository/RepositoryProtocol.swift:21:58: error: cannot find type 'Copyable' in scope
extension RepositoryPluckResult: Copyable where Wrapped: Copyable {
                                                         ^~~~~~~~
/Users/runner/work/mmex-ios/mmex-ios/MMEX/Repository/RepositoryProtocol.swift:21:34: error: cannot find type 'Copyable' in scope
extension RepositoryPluckResult: Copyable where Wrapped: Copyable {
                                 ^~~~~~~~

https://github.com/moneymanagerex/mmex-ios/actions/runs/11308990469/job/31452383557

@guanlisheng guanlisheng merged commit 8fc9287 into master Oct 14, 2024
1 check failed
@guanlisheng guanlisheng changed the title Enterview: show full category name with colon: delimiter Enterview: show full category name with delimiter Oct 14, 2024
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