Skip to content

Add initial README.md for Connections application - #2

Merged
Zacharymk1213 merged 9 commits into
Zacharymk1213:masterfrom
Benedek553:master
Aug 19, 2026
Merged

Add initial README.md for Connections application#2
Zacharymk1213 merged 9 commits into
Zacharymk1213:masterfrom
Benedek553:master

Conversation

@Benedek553

Copy link
Copy Markdown
Contributor

Added detailed documentation for the Connections application, including features, contact fields, project structure, requirements, and future ideas.

Important

Add a proper project license (e.g. LICENSE file and headers) before merging.

Added detailed documentation for the Connections application, including features, contact fields, project structure, requirements, and future ideas.
Copilot AI review requested due to automatic review settings July 4, 2026 18:14

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds an initial README.md documenting the Connections desktop contacts application (PyQt5 + SQLite), intended to help users understand features, data storage, and how to run the app.

Changes:

  • Introduces a comprehensive README covering features, UI screens, import flow, database behavior, and security notes.
  • Documents expected project layout and basic install/run instructions.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread README.md
Comment on lines +1 to +5
# Connections

A desktop contact management application written in Python using **PyQt5** and **SQLite**.

Connections allows you to organize contacts into multiple independent tables (groups), search across them, combine multiple datasets, and import contacts directly from Google Contacts CSV exports.
Comment thread README.md Outdated
Comment on lines +128 to +131
├── main.py
├── backend.py
├── global-network.ico
└── table.db
Comment thread README.md Outdated
### Windows

```
%LOCALAPPDATA%/connections-app/
Comment thread README.md Outdated
Install dependencies:

```bash
pip install PyQt5
Comment thread README.md Outdated
## Running

```bash
python main.py
@Zacharymk1213

Copy link
Copy Markdown
Owner

@Benedek553 ValueError: Received icon image '/Users/runner/work/Connections/Connections/global-network.ico' which exists but is not in the correct format. On this platform, only ('icns',) images may be used as icons. If Pillow is installed, automatic conversion will be attempted. Please install Pillow or convert your 'ico' file to one of ('icns',) and try again.

Add an icns file to the project so the make pipeline for Mac works.

@Benedek553

Copy link
Copy Markdown
Contributor Author

Added global-network.icns and updated the macOS PyInstaller workflow to use it instead of the Windows .ico file.

The existing global-network.ico is still kept for Windows builds.

@Zacharymk1213

Copy link
Copy Markdown
Owner

@Benedek553 Add a GPL V3 License and address the issues Copilot flagged above.

@Zacharymk1213

Copy link
Copy Markdown
Owner

@Benedek553 Apply the other fixes that can no longer be auto applied due to the latest commits eg. making sure the file structure is correct etc.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 3 out of 4 changed files in this pull request and generated 3 comments.

Comment thread README.md Outdated
Comment thread .github/workflows/cmake-single-platform-mac.yml Outdated
Comment thread README.md Outdated
Benedek553 and others added 5 commits July 6, 2026 17:03
@Benedek553

Benedek553 commented Aug 7, 2026

Copy link
Copy Markdown
Contributor Author

@Zacharymk1213 The Copilot suggested fixes now applied.

@Zacharymk1213
Zacharymk1213 merged commit 9996483 into Zacharymk1213:master Aug 19, 2026
3 checks 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.

3 participants