fix(linkedin): move session state pop before tabs; add rerun after settings merge
- Pop _linkedin_extracted before st.tabs() so tab_builder sees the freshly populated _parsed_resume in the same render pass (no extra rerun needed) - Fix tab label capitalisation: "Build Manually" (capital M) per spec - Add st.rerun() after LinkedIn merge in Settings so form fields refresh immediately to show the newly applied data
This commit is contained in:
parent
54d3d44f30
commit
a2778cc25d
2 changed files with 7 additions and 7 deletions
|
|
@ -270,8 +270,13 @@ elif step == 3:
|
||||||
st.subheader("Step 3 \u2014 Resume")
|
st.subheader("Step 3 \u2014 Resume")
|
||||||
st.caption("Upload your resume for fast parsing, or build it section by section.")
|
st.caption("Upload your resume for fast parsing, or build it section by section.")
|
||||||
|
|
||||||
|
# Read LinkedIn import result before tabs render (spec: "at step render time")
|
||||||
|
_li_data = st.session_state.pop("_linkedin_extracted", None)
|
||||||
|
if _li_data:
|
||||||
|
st.session_state["_parsed_resume"] = _li_data
|
||||||
|
|
||||||
tab_upload, tab_builder, tab_linkedin = st.tabs([
|
tab_upload, tab_builder, tab_linkedin = st.tabs([
|
||||||
"\U0001f4ce Upload", "\U0001f4dd Build manually", "\U0001f517 LinkedIn"
|
"\U0001f4ce Upload", "\U0001f4dd Build Manually", "\U0001f517 LinkedIn"
|
||||||
])
|
])
|
||||||
|
|
||||||
with tab_upload:
|
with tab_upload:
|
||||||
|
|
@ -362,12 +367,6 @@ elif step == 3:
|
||||||
)
|
)
|
||||||
|
|
||||||
with tab_linkedin:
|
with tab_linkedin:
|
||||||
# Check for pending LinkedIn import from previous rerun
|
|
||||||
_li_data = st.session_state.pop("_linkedin_extracted", None)
|
|
||||||
if _li_data:
|
|
||||||
st.session_state["_parsed_resume"] = _li_data
|
|
||||||
st.rerun() # re-render so tab_builder reads the newly populated _parsed_resume
|
|
||||||
|
|
||||||
from app.components.linkedin_import import render_linkedin_tab
|
from app.components.linkedin_import import render_linkedin_tab
|
||||||
render_linkedin_tab(config_dir=CONFIG_DIR, tier=_tier)
|
render_linkedin_tab(config_dir=CONFIG_DIR, tier=_tier)
|
||||||
|
|
||||||
|
|
|
||||||
|
|
@ -597,6 +597,7 @@ with tab_resume:
|
||||||
existing.update({k: v for k, v in _li_data.items() if v})
|
existing.update({k: v for k, v in _li_data.items() if v})
|
||||||
save_yaml(RESUME_PATH, existing)
|
save_yaml(RESUME_PATH, existing)
|
||||||
st.success("LinkedIn data applied to resume profile.")
|
st.success("LinkedIn data applied to resume profile.")
|
||||||
|
st.rerun()
|
||||||
|
|
||||||
with st.expander("🔗 Import from LinkedIn", expanded=False):
|
with st.expander("🔗 Import from LinkedIn", expanded=False):
|
||||||
from app.components.linkedin_import import render_linkedin_tab
|
from app.components.linkedin_import import render_linkedin_tab
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue